From: | tgl(at)postgresql(dot)org (Tom Lane) |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Fix WHERE CURRENT OF to work as designed within plpgsql. |
Date: | 2009-11-09 02:36:59 |
Message-ID: | 20091109023659.C1BAD753FB7@cvs.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Log Message:
-----------
Fix WHERE CURRENT OF to work as designed within plpgsql. The argument
can be the name of a plpgsql cursor variable, which formerly was converted
to $N before the core parser saw it, but that's no longer the case.
Deal with plain name references to plpgsql variables, and add a regression
test case that exposes the failure.
Modified Files:
--------------
pgsql/src/backend/executor:
execCurrent.c (r1.13 -> r1.14)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execCurrent.c?r1=1.13&r2=1.14)
pgsql/src/backend/parser:
gram.y (r2.688 -> r2.689)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/gram.y?r1=2.688&r2=2.689)
parse_expr.c (r1.247 -> r1.248)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/parse_expr.c?r1=1.247&r2=1.248)
pgsql/src/test/regress/expected:
plpgsql.out (r1.79 -> r1.80)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/plpgsql.out?r1=1.79&r2=1.80)
pgsql/src/test/regress/sql:
plpgsql.sql (r1.66 -> r1.67)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/plpgsql.sql?r1=1.66&r2=1.67)
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2009-11-09 15:18:10 | Re: [COMMITTERS] pgsql: Rewrite GEQO's gimme_tree function so that it always finds a |
Previous Message | Tom Lane | 2009-11-09 00:26:55 | pgsql: Modernize plpgsql's handling of parse locations, making it look a |