Re: remaining sql/json patches

From: Amit Langote <amitlangote09(at)gmail(dot)com>
To: jian he <jian(dot)universality(at)gmail(dot)com>
Cc: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Erik Rijkers <er(at)xs4all(dot)nl>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: remaining sql/json patches
Date: 2023-08-16 04:27:38
Message-ID: CA+HiwqHCSKqjR=KD5CaecTZNZEaTYg3AZEd4AWBuVi=G1+k9SQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Aug 15, 2023 at 5:58 PM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
> Hi.
> in v11, json_query:
> + The returned <replaceable>data_type</replaceable> has the
> same semantics
> + as for constructor functions like <function>json_objectagg</function>;
> + the default returned type is <type>text</type>.
> + The <literal>ON EMPTY</literal> clause specifies the behavior if the
> + <replaceable>path_expression</replaceable> yields no value at all; the
> + default when <literal>ON ERROR</literal> is not specified is
> to return a
> + null value.
>
> the default returned type is jsonb?

You are correct.

> Also in above quoted second last
> line should be <literal>ON EMPTY</literal> ?

Correct too.

> Other than that, the doc looks good.

Thanks for the review.

I will post a new version after finishing working on a few other
improvements I am working on.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Drouvot, Bertrand 2023-08-16 05:04:53 Re: WIP: new system catalog pg_wait_event
Previous Message Andres Freund 2023-08-16 04:16:53 Re: pg_logical_emit_message() misses a XLogFlush()