From: | Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: JSON path decimal literal syntax |
Date: | 2022-03-06 01:43:58 |
Message-ID: | 5c98c4ef-d6f1-fe60-2121-8594a982220b@postgrespro.ru |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 24.02.2022 21:24, Peter Eisentraut wrote:
> On 18.02.22 11:17, Peter Eisentraut wrote:
>> I noticed that the JSON path lexer does not support the decimal
>> literal syntax forms
>>
>> .1
>> 1.
>>
>> (that is, there are no digits before or after the decimal point).
>> This is allowed by the relevant ECMAScript standard
>> (https://262.ecma-international.org/5.1/#sec-7.8.3) and of course SQL
>> allows it as well.
>>
>> Is there a reason for this? I didn't find any code comments or
>> documentation about this.
>
> It has come to my attention that there are syntactic differences
> between JavaScript, which is what JSON path is built on, and JSON
> itself. Presumably, the JSON path lexer was originally built with the
> JSON syntax in mind.
>
> Attached is an updated patch that implements the JavaScript-based JSON
> path numeric literal syntax more correctly. Besides the above
> mentioned syntax forms, it now also rejects trailing junk after
> numeric literals more correctly, similar to how the main SQL lexer
> does it.
By the standard, jsonpath borrows its decimal literal syntax from the SQL,
so I think these lexer fixes are necessary.
Obviously, there are compatibility issues with expressions like
'1.type()', which will start to require parentheses around numbers,
but they seem to be useful only for our regression tests.
The corresponding changes in jsonpath_out() related to parentheses
are missing in the v2 patch:
=# select '(1).a'::jsonpath;
jsonpath
----------
1."a"
(1 row)
=# select '(1).a'::jsonpath::text::jsonpath;
ERROR: syntax error, unexpected STRING_P, expecting $end at or near """ of jsonpath input
I have added in v3 enclosing of numbers in parentheses if they have
successive path items. (Changed results of several test cases, one test
case added.)
--
Nikita Glukhov
Postgres Professional:http://www.postgrespro.com
The Russian Postgres Company
Attachment | Content-Type | Size |
---|---|---|
v3-0001-Make-JSON-path-numeric-literals-more-correct.patch | text/x-patch | 12.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Zhihong Yu | 2022-03-06 02:10:44 | timestamp for query in pg_stat_statements |
Previous Message | Paul Jungwirth | 2022-03-05 20:53:15 | Re: range_agg with multirange inputs |