| From: | "David E(dot) Wheeler" <david(at)justatheory(dot)com> |
|---|---|
| To: | Florents Tselai <florents(dot)tselai(at)gmail(dot)com> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(at)eisentraut(dot)org>, Robert Haas <robertmhaas(at)gmail(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> |
| Subject: | Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part |
| Date: | 2025-05-24 21:55:36 |
| Message-ID: | 8FD5F4CF-9059-40C1-9F0A-C3CBA8DFE17A@justatheory.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On May 24, 2025, at 17:46, David E. Wheeler <david(at)justatheory(dot)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
| Attachment | Content-Type | Size |
|---|---|---|
| v4-0001-Add-additional-jsonpath-string-methods.patch | application/octet-stream | 51.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David E. Wheeler | 2025-05-24 22:01:21 | Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part |
| Previous Message | David E. Wheeler | 2025-05-24 21:46:49 | Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part |