From: | Florents Tselai <florents(dot)tselai(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "David E(dot) Wheeler" <david(at)justatheory(dot)com>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter(at)eisentraut(dot)org> |
Subject: | Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part |
Date: | 2025-05-22 16:38:02 |
Message-ID: | 9E77459B-0E6D-4105-A8D9-E5AED5F0AC0B@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 22 May 2025, at 5:05 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>
> On Wed, May 21, 2025 at 2:31 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Having said that, what's wrong with inventing some improved function
>> names and never removing the old ones?
>
> I don't particularly like the clutter, but if the consensus is that
> the clutter doesn't matter, fair enough.
>
It depends really on how much future work we expect in adding more methods in jsonpath.
I think there’s a lot of potential there, but that’s a guess really.
On David’s point about popularity:
In my experience timestamp related stuff from jsonb documents end up in a generated column,
and are indexed & queried there.
I expect that to continue in PG18 onwards as we’ll have virtual gen columns too.
Just to be clear, though, adding another version of these functions means
we’ll have an additional (now third) set of the same 5 functions:
The vanilla versions are considered stable and the suffixed *_tz or *_volatile (?)
jsonb_path_exists
jsonb_path_query
jsonb_path_query_array
jsonb_path_query_first
jsonb_path_match
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2025-05-22 16:45:25 | Re: [Util] Warn and Remove Invalid GUCs |
Previous Message | Shaik Mohammad Mujeeb | 2025-05-22 16:14:04 | Re: [Util] Warn and Remove Invalid GUCs |