Horton
Menu

Re: More jsonpath methods: translate, split, join

Florents Tselai <florents.tselai@gmail.com>

View thread
From: Florents Tselai <florents.tselai@gmail.com>
To: Zsolt Parragi <zsolt.parragi@percona.com>
Cc: pgsql-hackers@lists.postgresql.org, Corey Huinker <corey.huinker@gmail.com>
Date: 2026-07-04T08:59:09Z
Lists: pgsql-hackers

Attachments

On Sat, Jun 27, 2026 at 8:40 AM Florents Tselai <florents.tselai@gmail.com>
wrote:

>
>
> On Fri, Jun 26, 2026 at 11:19 PM Zsolt Parragi <zsolt.parragi@percona.com>
> wrote:
>
>> +                                       binary_jbv.type = jbvBinary;
>> +                                       binary_jbv.val.binary.data =
>> &jb_result->root;
>> +                                       binary_jbv.val.binary.len =
>> VARSIZE(jb_result);
>>
>> Shouldn't that use VARSIZE_ANY_EXHDR or VARSIZE(jb_result) - VARHDRSZ?
>>
>
> I think you're right; let's see if Corey comes back with anything else
> before I send out a new version.
>

Here's a v5 that fixes the  VARSIZE_ANY_EXHDR Zsolt mentioned above
I've also rearranged the test cases in jsonpath.sql and jsonb_jsonpath.sql
Also added parser error test cases that were missing for translate() in
jsonpath.sql