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: jian he <jian.universality@gmail.com>

Cc: Chao Li <li.evan.chao@gmail.com>, Álvaro Herrera <alvherre@kurilemu.de>, Florents Tselai <florents.tselai@gmail.com>, 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-11-30T21:16:34Z

Lists: pgsql-hackers

Attachments

v15-0001-Rename-jsonpath-method-arg-tokens.patch (application/octet-stream)
v15-0002-Add-additional-jsonpath-string-methods.patch (application/octet-stream)

On Nov 28, 2025, at 05:29, jian he <jian.universality@gmail.com> wrote:

> some "switch" in the attached patch does not preserve the JsonPathItemType order
> consistency, like executeItemOptUnwrapTarget.

Well-spotted, thank you! Fixed in v15, attached.

Best,

David