Re: Patch: Improve Boolean Predicate JSON Path Docs - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Patch: Improve Boolean Predicate JSON Path Docs
Date
Msg-id 365431.1706198604@sss.pgh.pa.us
Whole thread Raw
In response to Re: Patch: Improve Boolean Predicate JSON Path Docs  ("David E. Wheeler" <david@justatheory.com>)
Responses Re: Patch: Improve Boolean Predicate JSON Path Docs
List pgsql-hackers
"David E. Wheeler" <david@justatheory.com> writes:
> On Jan 24, 2024, at 16:32, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> +      <para>
>> +       Predicate check expressions are required in the
>> +       <literal>@@</literal> operator (and the
>> +       <function>jsonb_path_match</function> function), and should not be used
>> +       with the <literal>@?</literal> operator (or the
>> +       <function>jsonb_path_exists</function> function).
>> +      </para>
>> +     </note>
>> +    </sect4>

> I had this bit here:

>       <para>
>        Conversely, non-predicate <type>jsonpath</type> expressions should not be
>        used with the <literal>@@</literal> operator (or the
>        <function>jsonb_path_match</function> function).
>       </para>

I changed the preceding para to say "... check expressions are
required in ...", which I thought was sufficient to cover that.
Also, the tabular description of the operator tells you not to do it.

> What do you think of also dropping the article from all the references to “the strict mode” or “the lax mode”, to
makethem “strict mode” and “lax mode”, respectively? 

Certainly most of 'em don't need it.  I'll make it so.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: A compiling warning in jsonb_populate_record_valid
Next
From: Peter Eisentraut
Date:
Subject: Re: make dist using git archive