pgsql: Re-allow duplicate aliases within aliased JOINs. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Re-allow duplicate aliases within aliased JOINs.
Date
Msg-id E1Vftdv-0002tG-JD@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Re-allow duplicate aliases within aliased JOINs.

Although the SQL spec forbids duplicate table aliases, historically
we've allowed queries like
    SELECT ... FROM tab1 x CROSS JOIN (tab2 x CROSS JOIN tab3 y) z
on the grounds that the aliased join (z) hides the aliases within it,
therefore there is no conflict between the two RTEs named "x".  The
LATERAL patch broke this, on the misguided basis that "x" could be
ambiguous if tab3 were a LATERAL subquery.  To avoid breaking existing
queries, it's better to allow this situation and complain only if
tab3 actually does contain an ambiguous reference.  We need only remove
the check that was throwing an error, because the column lookup code
is already prepared to handle ambiguous references.  Per bug #8444.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/648bd05b13b3624e494ae2996c2d2e0241cefe87

Modified Files
--------------
src/backend/parser/parse_clause.c   |    5 +++--
src/backend/parser/parse_relation.c |   15 +++++++++++++--
src/test/regress/expected/join.out  |   24 ++++++++++++++++++++++++
src/test/regress/sql/join.sql       |   12 ++++++++++++
4 files changed, 52 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Re-allow duplicate aliases within aliased JOINs.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix ruleutils pretty-printing to not generate trailing whitespac