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.
--
Thanks, Amit Langote