Re: ExecTypeSetColNames is fundamentally broken
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-03-15T20:23:56Z
Lists: pgsql-hackers
On Tue, Dec 7, 2021 at 1:19 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > So the alternatives I see are to revert what bf7ca1587 tried to do > here, or to try to make it work that way across-the-board, which > implies (a) a very much larger amount of work, and (b) breaking > important behaviors that are decades older than that commit. > It's not even entirely clear that we could get to complete > consistency if we went down that path. Continuing my tour through the "bug fixes" section of the CommitFest, I came upon this thread. Unfortunately there's not that much I can do to progress it, because I've already expressed all the opinions that I have on this thread. If we back-patch Tom's originally proposed fix, i expect we might get a complaint or too, but the current behavior of being able to create unreadable tables is indisputably poor, and I'm not in a position to tell Tom that he has to go write a different fix instead, or even that such a fix is possible. Unless somebody else wants to comment, which IMHO would be good, I think it's up to Tom to make a decision here on how he'd like to proceed and then, probably, just do it. Anyone else have thoughts? -- Robert Haas EDB: http://www.enterprisedb.com
Commits
-
Don't bother to attach column name lists to RowExprs of named types.
- d7b5c071dd6a 15.0 landed
-
Revert applying column aliases to the output of whole-row Vars.
- ec62cb0aac5b 15.0 landed
- 5e144cc89b46 13.7 landed
- 205214c8b8cc 10.21 landed
- 1f5ef5ae0806 12.11 landed
- 1d072bd2030a 14.3 landed
- 13b54d1e0dde 11.16 landed