Re: Patch bug: Fix jsonpath .* on Arrays

Поиск
Список
Период
Сортировка
От Степан Неретин
Тема Re: Patch bug: Fix jsonpath .* on Arrays
Дата
Msg-id 1719290785.518941016@f456.i.mail.ru
обсуждение исходный текст
Ответ на Re: Patch bug: Fix jsonpath .* on Arrays  ("David E. Wheeler" <david@justatheory.com>)
Ответы Re: Patch bug: Fix jsonpath .* on Arrays
Список pgsql-hackers


 
Вторник, 25 июня 2024, 11:17 +07:00 от David E. Wheeler <david@justatheory.com>:
 
On Jun 7, 2024, at 10:23, David E. Wheeler <david@justatheory.com> wrote:
 
> Rebased and moved the new tests to the end of the file.

Bah, sorry, that was the previous patch. Here’s v3.

D
 
 
 
Hi! Looks good to me, but I have several comments.
Your patch improves tests, but why did you change formatting in jsonpath_exec.c? What's the motivation?
 
[1] select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'strict $.*');
I propose adding a similar test with explicitly specified lax mode: select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'lax $.*'); to show what lax mode is set by default.
 
Odd empty result for the test: select jsonb '[1,2,3,{"b": [3,4,5]}]' @? 'strict $.*';
I expected an error like in test [1]. This behavior is not obvious to me.
 
Everything else is cool. Thanks to the patch and the discussion above, I began to understand better how wildcards in JSON work.

Best regards, Stepan Neretin.
 

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Amit Langote
Дата:
Сообщение: Re: sql/json miscellaneous issue
Следующее
От: Stan Hu
Дата:
Сообщение: Re: [PATCH] Fix type redefinition build errors with macOS SDK 15.0