Re: Bug in jsonb_path_exists (maybe _match) one-element scalar/variable jsonpath handling

On Fri, Dec 2, 2022 at 3:18 PM Alexander Korotkov <aekorotkov(at)gmail(dot)com> wrote:
> Draft patch fixing the issue is attached. Let me know what you think
> about this.

Revised patch is attached, wrong pfree() is fixed. I was intended to
backpatch it. But the behavior change makes me uneasy.

select * from jsonb_path_query('{"a": 10}', '$ ? (@.a < $value)');

Currently, this query generates an error because of missing "value"
variable. The patch suppress this error. I'm not sure this error
should be suppressed. Especially, I'm sure this should be
backpatched.

Should we fix only existence checking behaviour and let other cases
throw an error? Thoughts?

------
Regards,
Alexander Korotkov