From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Jeevan Chalke <jeevan(dot)chalke(at)enterprisedb(dot)com>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
Cc: | tgl(at)sss(dot)pgh(dot)pa(dot)us, peter(at)eisentraut(dot)org, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: More new SQL/JSON item methods |
Date: | 2024-02-27 07:10:12 |
Message-ID: | 503e46de-b5b5-b769-fe7f-ee2a9f046d3e@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2024-02-02 Fr 00:31, Jeevan Chalke wrote:
>
>
> On Thu, Feb 1, 2024 at 11:25 AM Kyotaro Horiguchi
> <horikyota(dot)ntt(at)gmail(dot)com> wrote:
>
> At Thu, 1 Feb 2024 09:22:22 +0530, Jeevan Chalke
> <jeevan(dot)chalke(at)enterprisedb(dot)com> wrote in
> > On Thu, Feb 1, 2024 at 7:24 AM Kyotaro Horiguchi
> <horikyota(dot)ntt(at)gmail(dot)com>
> > wrote:
> >
> > > At Thu, 01 Feb 2024 10:49:57 +0900 (JST), Kyotaro Horiguchi <
> > > horikyota(dot)ntt(at)gmail(dot)com> wrote in
> > > > By the way, while playing with this feature, I noticed the
> following
> > > > error message:
> > > >
> > > > > select jsonb_path_query('1.1' , '$.boolean()');
> > > > > ERROR: numeric argument of jsonpath item method
> .boolean() is out of
> > > range for type boolean
> > > >
> > > > The error message seems a bit off to me. For example,
> "argument '1.1'
> > > > is invalid for type [bB]oolean" seems more appropriate for this
> > > > specific issue. (I'm not ceratin about our policy on the
> spelling of
> > > > Boolean..)
> > >
> > > Or, following our general convention, it would be spelled as:
> > >
> > > 'invalid argument for type Boolean: "1.1"'
> > >
> >
> > jsonpath way:
>
> Hmm. I see.
>
> > ERROR: argument of jsonpath item method .boolean() is invalid
> for type
> > boolean
> >
> > or, if we add input value, then
> >
> > ERROR: argument "1.1" of jsonpath item method .boolean() is
> invalid for
> > type boolean
> >
> > And this should work for all the error types, like out of range,
> not valid,
> > invalid input, etc, etc. Also, we don't need separate error
> messages for
> > string input as well, which currently has the following form:
> >
> > "string argument of jsonpath item method .%s() is not a valid
> > representation.."
>
> Agreed.
>
>
> Attached are patches based on the discussion.
Thanks, I combined these and pushed the result.
cheers
andrew
--
Andrew Dunstan
EDB:https://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Smith | 2024-02-27 07:17:44 | Re: Synchronizing slots from primary to standby |
Previous Message | Michael Paquier | 2024-02-27 06:25:35 | Re: Improve readability by using designated initializers when possible |