From: | Amit Langote <amitlangote09(at)gmail(dot)com> |
---|---|
To: | jian he <jian(dot)universality(at)gmail(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: sql/json remaining issue |
Date: | 2024-04-26 03:34:10 |
Message-ID: | CA+HiwqHDzLWOgBu4jA1yQk165zQQ_y9PeoTvtY=JO7SUisN72Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Apr 18, 2024 at 9:33 AM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> On Mon, Apr 15, 2024 at 9:46 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > On Sat, Apr 13, 2024 at 11:12 PM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
> > > On Fri, Apr 12, 2024 at 5:44 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > > >
> > > > > elog(ERROR, "unrecognized json wrapper %d", wrapper);
> > > > > should be
> > > > > elog(ERROR, "unrecognized json wrapper %d", (int) wrapper);
> > > >
> > > > Fixed in 0003.
> > > >
> > > the fix seems not in 0003?
> > > other than that, everything looks fine.
>
> Oops, really fixed now in 0002.
>
> > I've combined these patches into one -- attached 0001. Will push tomorrow.
>
> Decided to break the error message improvement patch into its own
> after all -- attached 0001.
>
> > Now studying the JsonBehavior DEFAULT expression issue and your patch.
>
> I found some more coercion-related expression nodes that must also be
> checked along with CoerceViaIO and CoerceToDomain. Also, after fixing
> the code to allow them, I found that we'd need to also check
> recursively whether their argument expression is also one of the
> supported expression nodes. Also, I decided that it's not necessary
> to include "cast" in the error message as one of the supported
> expressions.
>
> Will push all today.
Totally forgot to drop a note here that I pushed those and marked the
2 open items as resolved.
--
Thanks, Amit Langote
From | Date | Subject | |
---|---|---|---|
Next Message | Kashif Zeeshan | 2024-04-26 05:16:37 | Re: Help update PostgreSQL 13.12 to 13.14 |
Previous Message | Corey Huinker | 2024-04-26 03:27:08 | Re: Statistics Import and Export |