]> granicus.if.org Git - postgresql/commit
Fix ruleutils to cope with conflicts from adding/dropping/renaming columns.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 31 Dec 2012 20:13:26 +0000 (15:13 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 31 Dec 2012 20:13:26 +0000 (15:13 -0500)
commit2ffa740be9d96a3743ecb7e42391c53d0760c65a
treea84495c924796e77581669083ce1559a8b743dc7
parent7eb559a86d773e0105ae5823e0c776c3af3c3fe2
Fix ruleutils to cope with conflicts from adding/dropping/renaming columns.

In commit 11e131854f8231a21613f834c40fe9d046926387, we improved the
rule/view dumping code so that it would produce valid query representations
even if some of the tables involved in a query had been renamed since the
query was parsed.  This patch extends that idea to fix problems that occur
when individual columns are renamed, or added or dropped.  As before, the
core of the fix is to assign unique new aliases when a name conflict has
been created.  This is complicated by the JOIN USING feature, which
requires the same column alias to be used in both input relations, but we
can handle that with a sufficiently complex approach to assigning aliases.

A fortiori, this patch takes care of situations where the query didn't have
unique column names to begin with, such as in a recent complaint from Bryan
Nuse.  (Because of expansion of "SELECT *", re-parsing a dumped query can
require column name uniqueness even though the original text did not.)
src/backend/commands/explain.c
src/backend/commands/typecmds.c
src/backend/utils/adt/ruleutils.c
src/test/regress/expected/create_view.out
src/test/regress/sql/create_view.sql