public inbox for [email protected]
help / color / mirror / Atom feedFrom: Tom Lane <[email protected]>
To: Aleksander Alekseev <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Cc: Robert Haas <[email protected]>
Subject: Re: ExecTypeSetColNames is fundamentally broken
Date: Thu, 17 Mar 2022 18:28:11 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAJ7c6TPfqfvB8AShZir1R054Wn7M5G4i_SgRM9K9-kADys8qEg@mail.gmail.com>
References: <[email protected]>
<CA+TgmoaYDAqLLmxo_d9h2SZ1Mw28M9w5QZMuZcaL6D2JdG_B6w@mail.gmail.com>
<[email protected]>
<CA+Tgmoa2Fm+4eDb6O4NFzA-u78k2Wa0-V48qimSVgBbCF2MvDg@mail.gmail.com>
<[email protected]>
<CA+TgmoYX8ZsL5BoYeJD+QK9iCQLBdq1mbQ_ZiwZGKBwzgFZYWA@mail.gmail.com>
<[email protected]>
<CA+TgmoZx+_6VqwZmG4Re5geiC=Jas=mAc0Jjve=b0AHVWi4q3g@mail.gmail.com>
<CAJ7c6TPfqfvB8AShZir1R054Wn7M5G4i_SgRM9K9-kADys8qEg@mail.gmail.com>
Aleksander Alekseev <[email protected]> writes:
> I understand the concern expressed by Robert in respect of backward
> compatibility. From the user's perspective, personally I would prefer
> a fixed bug over backward compatibility. Especially if we consider the
> fact that the current behaviour of cases like `select row_to_json(i)
> from int8_tbl i(x,y)` is not necessarily the correct one, at least it
> doesn't look right to me.
It's debatable anyway. I'd be less worried about changing this behavior
if we didn't have to back-patch it ... but I see no good alternative.
> So unless anyone has strong objections against the proposed fix or can
> propose a better patch, I would suggest merging it.
Done now.
regards, tom lane
view thread (2+ messages)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: ExecTypeSetColNames is fundamentally broken
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox