| From: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Jeff Janes <jeff(dot)janes(at)gmail(dot)com>, Erwin Brandstetter <brsaweda(at)gmail(dot)com>, PostgreSQL Bug List <pgsql-bugs(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Variable substitution in jsonb functions fails for jsonpath operator like_regex |
| Date: | 2023-10-19 02:13:16 |
| Message-ID: | CAKFQuwaBB+PPZkn0GtTzt5z6Vtypt2CQ-5+nK7XzW6zm9UqaDQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
On Wed, Oct 18, 2023, 18:47 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Jeff Janes <jeff(dot)janes(at)gmail(dot)com> writes:
> > On Wed, Oct 18, 2023 at 5:20 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >> Maybe it wouldn't take more than s/STRING_P/starts_with_initial/
> >> here (though I'd then rename starts_with_initial to something
> >> more generic).
>
> > I tried it, and it didn't work. No error, it just doesn't match
> > anything--including literal values which do match things in HEAD.
> Oh, duh, makeItemLikeRegex() is expecting a plain JsonPathString,
> and JsonPathParseItem.value.like_regex is set up for just a
> constant pattern. We could no doubt fix that, but it's a bigger
> lift than I was hoping.
>
Not in a position to do it myself but we should confirm we aren't simply
following the standard here, and if so decide whether we want to deviate.
David J.
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Aleš Zelený | 2023-10-19 06:43:43 | Re: Invalid dump file after drop of role that previously created extension containing a table. |
| Previous Message | Tom Lane | 2023-10-19 01:46:29 | Re: Variable substitution in jsonb functions fails for jsonpath operator like_regex |