On Tue, Jul 9, 2024 at 12:30 PM Amit Langote <amitlangote09@gmail.com> wrote:
> On Tue, Jul 9, 2024 at 10:39 AM jian he <jian.universality@gmail.com> wrote:
> > On Mon, Jul 8, 2024 at 8:57 PM Amit Langote <amitlangote09@gmail.com> wrote:
> > >
> > > Updated patch attached.
> > >
> >
> > Returns true if the SQL/JSON <replaceable>path_expression</replaceable>
> > - applied to the <replaceable>context_item</replaceable> using the
> > - <literal>PASSING</literal> <replaceable>value</replaceable>s yields any
> > - items.
> > + applied to the <replaceable>context_item</replaceable> doesn't yield
> > + any items.
> > should "doesn't" be removed?
> > should it be "yields"?
>
> Oops, fixed.
>
> > + set. The <literal>ON ERROR</literal> clause specifies the behavior
> > + if an error occurs when evaluating
> > <replaceable>path_expression</replaceable>,
> > + when coercing the result value to the
> > <literal>RETURNING</literal> type,
> > + or when evaluating the <literal>ON EMPTY</literal> expression if the
> > + <replaceable>path_expression</replaceable> evaluation results in an
> > + empty set.
> > last sentence, "in an empty set." should be "is an empty set"
>
> "results in an empty set" here means "the result of the evaluation is
> an empty set", similar to:
>
> $ git grep "results in an" doc
> doc/src/sgml/charset.sgml: results in an error, because even though
> the <literal>||</literal> operator
> doc/src/sgml/plpgsql.sgml: an omitted <literal>ELSE</literal>
> clause results in an error rather
> doc/src/sgml/plpython.sgml: If the second <literal>UPDATE</literal>
> statement results in an
> doc/src/sgml/pltcl.sgml: If the second <command>UPDATE</command>
> statement results in an
>
> Maybe I could just replace that by "returns an empty set".
>
> Will push shortly after making those changes.
And...pushed. Thanks for the reviews.
--
Thanks, Amit Langote