Re: Doc Rework: Section 9.16.13 SQL/JSON Query Functions - Mailing list pgsql-hackers

From Amit Langote
Subject Re: Doc Rework: Section 9.16.13 SQL/JSON Query Functions
Date
Msg-id CA+HiwqEPSh0zG9LFOOWvFBMRocAcDaUTUmTupq97xeQ+CESG4Q@mail.gmail.com
Whole thread Raw
In response to Re: Doc Rework: Section 9.16.13 SQL/JSON Query Functions  (Erik Rijkers <er@xs4all.nl>)
List pgsql-hackers
On Fri, Jul 5, 2024 at 10:16 PM Erik Rijkers <er@xs4all.nl> wrote:
> Op 7/5/24 om 14:35 schreef Amit Langote:
> > Hi Jian,
> >
> > Thanks for the reviews.
> >
>  > [v3-0001-SQL-JSON-Various-improvements-to-SQL-JSON-query-f.patch]
>        i.e., from the patch for doc/src/sgml/func.sgml
>
>
> Small changes:
>
> 4x:
> 'a SQL'  should be
> 'an SQL'
> ('a SQL' does never occur in the docs; it's always 'an SQL'; apperently
> the 'sequel' pronunciation is out)
>
> 'some other type to which can be successfully coerced'
> 'some other type to which it can be successfully coerced'
>
>
> 'specifies the behavior behavior'
> 'specifies the behavior'
>
>
> In the following sentence:
>
> "By default, the result is returned as a value of type <type>jsonb</type>,
> though the <literal>RETURNING</literal> clause can be used to return
> the original <type>jsonb</type> value as some other type to which it
> can be successfully coerced."
>
> it seems to me that this phrase is better removed:
>     "the original <type>jsonb</type> value as"

Thanks, I've addressed all these in the next patch I'll send.

--
Thanks, Amit Langote



pgsql-hackers by date:

Previous
From: ikedarintarof
Date:
Subject: Re: doc: modify the comment in function libpqrcv_check_conninfo()
Next
From: Amit Kapila
Date:
Subject: Re: Slow catchup of 2PC (twophase) transactions on replica in LR