public inbox for [email protected]
help / color / mirror / Atom feedFrom: Richard Guo <[email protected]>
To: Tom Lane <[email protected]>
Cc: Amit Langote <[email protected]>
Cc: Vik Fearing <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: Álvaro Herrera <[email protected]>
Cc: Nathan Bossart <[email protected]>
Cc: Heikki Linnakangas <[email protected]>
Cc: Melanie Plageman <[email protected]>
Subject: Re: BUG #19418: SQL/JSON JSON_VALUE() does not conform to ISO/IEC 9075-2:2023(E) 6.34 <JSON value constructor>
Date: Thu, 23 Apr 2026 09:55:10 +0900
Message-ID: <CAMbWs49ps9eGWvkwc64ytUxdK0s-=xdFjtf64WyGp65EgTZ_LQ@mail.gmail.com> (raw)
In-Reply-To: <CAMbWs4_vqNEcHpvKFeFxfRoisrvw6yJ3styhS-arQDHZirw5cQ@mail.gmail.com>
References: <[email protected]>
<[email protected]>
<CAMbWs4_4Zc7O4pCU_nJU_8=Y2bOS3sEXJR=WH39Kc__UuaCW3w@mail.gmail.com>
<CAMbWs4_f-4BqvfKTt0YbkOky+Gf0Mpi2kCT3CAE8brZFnK275Q@mail.gmail.com>
<CAMbWs4-y4CEy66yA-Hh+6Snqbok3XTMFj5h87gUk_15Hhjd3qw@mail.gmail.com>
<[email protected]>
<CAMbWs49P2z40f5yb_97ErwtC9ocdR5QQN0gWUZtjgot9+iGAcg@mail.gmail.com>
<CAMbWs4-NuYP+0Zq9=Ufwy5GcM0XQnEqWjJFucaRGO4EQgiCcow@mail.gmail.com>
<CAMbWs4-ra9mkBhUs2kmB5KxWuo80oAA33gZ=8JmRxmG3qGNxUg@mail.gmail.com>
<CA+HiwqEtiTNV2v2P0HGa0B1TMNfmPweKXMZEOYPoQuFf3ZhsXQ@mail.gmail.com>
<CAMbWs49tDE_niKLr4mzsa2BkX7fkWrorBzWheKTBziZ7z9-AuQ@mail.gmail.com>
<CAMbWs48KEdgyJhxCOysaRdRJ5NzPObUET+5XbvyiqDfqyABEwg@mail.gmail.com>
<[email protected]>
<CAMbWs4_vqNEcHpvKFeFxfRoisrvw6yJ3styhS-arQDHZirw5cQ@mail.gmail.com>
On Tue, Apr 21, 2026 at 12:20 PM Richard Guo <[email protected]> wrote:
>
> On Tue, Apr 21, 2026 at 11:30 AM Tom Lane <[email protected]> wrote:
> > Richard Guo <[email protected]> writes:
> > > Another question I'd like to raise: is it OK to commit this patch to
> > > master given that feature freeze has passed? I think the answer is
> > > yes, because this is arguably a bug fix rather than a new feature.
> > > However, it does change user-visible behavior, and existing app code
> > > that relies on the NULL behavior would break. So if we commit it, we
> > > need to add in the release notes about this incompatibility.
>
> > Well, if we definitely intend to commit a compatibility-breaking
> > change, I think it's better to commit it sooner not later. If we
> > wait till v20, all we accomplish is to give users another year to
> > write code that depends on the old behavior.
> >
> > However, usually at this stage of the cycle the answer to such
> > questions is "let the RMT decide". Take the question to them
> > (cc'd).
>
> Thanks Tom for the suggestion.
Not sure the RMT mailing list works or not, maybe I'd better CC RMT
members.
- Richard
> Hi RMT,
>
> I'd like to commit a fix for JSON_ARRAY(subquery) behavior that
> involves a user-visible incompatibility, and would appreciate your
> go/no-go since we're past feature freeze.
>
> Summary:
>
> - JSON_ARRAY(SELECT ...) currently returns NULL over an empty result
> set, but the SQL/JSON standard requires it to return '[]'. Fixing
> this changes user-visible output.
>
> - The same patch also fixes a deparsing issue: views defined with
> JSON_ARRAY(SELECT ...) are dumped back as the internal JSON_ARRAYAGG
> rewrite instead of the original syntax.
>
> - Richard
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], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: BUG #19418: SQL/JSON JSON_VALUE() does not conform to ISO/IEC 9075-2:2023(E) 6.34 <JSON value constructor>
In-Reply-To: <CAMbWs49ps9eGWvkwc64ytUxdK0s-=xdFjtf64WyGp65EgTZ_LQ@mail.gmail.com>
* 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