Re: dubious warning: FORMAT JSON has no effect for json and jsonb types - Mailing list pgsql-hackers

From Amit Langote
Subject Re: dubious warning: FORMAT JSON has no effect for json and jsonb types
Date
Msg-id CA+HiwqECx-un1Qgr8JC0Eg4_UuKJqYTHOgWZh1XiZJag3Bx0Tg@mail.gmail.com
Whole thread Raw
In response to Re: dubious warning: FORMAT JSON has no effect for json and jsonb types  (Peter Eisentraut <peter@eisentraut.org>)
List pgsql-hackers
On Fri, Aug 18, 2023 at 2:59 PM Peter Eisentraut <peter@eisentraut.org> wrote:
> On 16.08.23 16:59, Merlin Moncure wrote:
> > On Wed, Aug 16, 2023 at 8:55 AM Peter Eisentraut <peter@eisentraut.org
> > <mailto:peter@eisentraut.org>> wrote:
> >
> >     This warning comes from parse_expr.c transformJsonValueExpr() and is
> >     triggered for example by the following test case:
> >
> >     SELECT JSON_OBJECT('foo': NULL::json FORMAT JSON);
> >     WARNING:  FORMAT JSON has no effect for json and jsonb types
> >
> >     But I don't see anything in the SQL standard that would require this
> >     warning.  It seems pretty clear that FORMAT JSON in this case is
> >     implicit and otherwise without effect.
> >
> >     Also, we don't have that warning in the output case (RETURNING json
> >     FORMAT JSON).
> >
> >     Anyone remember why this is here?  Should we remove it?
> >
> >
> > +1 for removing, on the basis that it is not suprising, and would
> > pollute logs for most configurations.
>
> done

+1 and thanks.  May have been there as a debugging aid if anything.

--
Thanks, Amit Langote
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Remove distprep
Next
From: "Drouvot, Bertrand"
Date:
Subject: Re: WIP: new system catalog pg_wait_event