]> granicus.if.org Git - postgresql/commit
Change plpgsql from using textual substitution to insert variable references
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 6 Nov 2009 18:37:55 +0000 (18:37 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 6 Nov 2009 18:37:55 +0000 (18:37 +0000)
commit0772f1e53d2b10021a1c1513a8d44e2eb137a570
tree05632d2d8cbfb60df46d1edaab8865ddc0ff7d70
parent593f4b854a8bb384547b8fa9854c73dcd88d4876
Change plpgsql from using textual substitution to insert variable references
into SQL expressions, to using the newly added parser callback hooks.

This allows us to do the substitutions in a more semantically-aware way:
a variable reference will only be recognized where it can validly go,
ie, a place where a column value or parameter would be legal, instead of
the former behavior that would replace any textual match including
table names and column aliases (leading to syntax errors later on).
A release-note-worthy fine point is that plpgsql variable names that match
fully-reserved words will now need to be quoted.

This commit preserves the former behavior that variable references take
precedence over any possible match to a column name.  The infrastructure
is in place to support the reverse precedence or throwing an error on
ambiguity, but those behaviors aren't accessible yet.

Most of the code changes here are associated with making the namespace
data structure persist so that it can be consulted at runtime, instead
of throwing it away at the end of initial function parsing.

The plpgsql scanner is still doing name lookups, but that behavior is
now irrelevant for SQL expressions.  A future commit will deal with
removing unnecessary lookups.
src/pl/plpgsql/src/gram.y
src/pl/plpgsql/src/pl_comp.c
src/pl/plpgsql/src/pl_exec.c
src/pl/plpgsql/src/pl_funcs.c
src/pl/plpgsql/src/plpgsql.h
src/test/regress/expected/plpgsql.out
src/test/regress/sql/plpgsql.sql