Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part

David E. Wheeler <david@justatheory.com>

From: "David E. Wheeler" <david@justatheory.com>

To: Florents Tselai <florents.tselai@gmail.com>

Cc: Tom Lane <tgl@sss.pgh.pa.us>, Peter Eisentraut <peter@eisentraut.org>, Robert Haas <robertmhaas@gmail.com>, Alexander Korotkov <aekorotkov@gmail.com>, pgsql-hackers <pgsql-hackers@lists.postgresql.org>, Andrew Dunstan <andrew@dunslane.net>

Date: 2025-05-24T21:55:36Z

Lists: pgsql-hackers

Attachments

v4-0001-Add-additional-jsonpath-string-methods.patch (application/octet-stream) patch v4-0001

On May 24, 2025, at 17:46, David E. Wheeler <david@justatheory.com> wrote:

> Oh, and I forgot to mention, src/backend/utils/adt/jsonpath_scan.l looks like it has a ton of duplication in it. Shouldn’t it just add the new keywords, something like:

And now I see my patch broke the grammar because I left some of my fiddling in there. Apologies. Here’s an updated patch with the updated keyword map, too.

Best,

David

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Add additional jsonpath string methods
- bd4f879a9cdd 19 (unreleased) landed
Rename jsonpath method arg tokens
- a35c9d524ed0 19 (unreleased) landed
Fix transient memory leakage in jsonpath evaluation.
- 5a2043bf7131 19 (unreleased) cited
Make jsonpath .string() be immutable for datetimes.
- cb599b9ddfcc 18.0 cited