Content-Type: text/plain;
	charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3864.500.181\))
Subject: Re: BUG #19458: OOM killer in jsonb_path_exists_opr (@?) with
 malformed JSONPath containing non-existent variables
From: Andrey Borodin <x4mmm@yandex-team.ru>
In-Reply-To: <CAN-LCVNh2z4EE+F21XPe7XRSWfPFtZx1WYAswpU5qs+RdR=jjg@mail.gmail.com>
Date: Thu, 30 Apr 2026 17:03:29 +0500
Cc: Amit Langote <amitlangote09@gmail.com>,
 Andrey Rachitskiy <pl0h0yp1@gmail.com>,
 pgsql-bugs@lists.postgresql.org,
 dhyan@nataraj.su
Content-Transfer-Encoding: quoted-printable
Message-Id: <6DE4A9D6-6D5C-41C0-8AFE-F51CFBDDAD5A@yandex-team.ru>
References: <19458-a69c98bc498333ba@postgresql.org>
 <CAB8bMit1HvJsAasUYwmq+82Oa3zQhJyvsHNS4PGF_S_BCMnuVA@mail.gmail.com>
 <AE0E8193-85BD-4A10-BE23-582D92DA281D@yandex-team.ru>
 <CAN-LCVNh2z4EE+F21XPe7XRSWfPFtZx1WYAswpU5qs+RdR=jjg@mail.gmail.com>
To: Nikita Malakhov <hukutoc@gmail.com>
Archived-At: <https://www.postgresql.org/message-id/6DE4A9D6-6D5C-41C0-8AFE-F51CFBDDAD5A%40yandex-team.ru>
Precedence: bulk


> On 29 Apr 2026, at 01:19, Nikita Malakhov <hukutoc@gmail.com> wrote:
>=20
> According to the Jsonpath standard, malformed expression should return =
an error,
> but not all cases of malformation are thoroughly described.
>=20
> When this functionality was developed (Jsonpath and SQL/JSON) the =
absence
> of the variable was considered as malformation and was decided to =
throw an error
> in threads long time ago. In case this behavior to be a subject for =
change it surely
> should not be backported, but the error-throwing code has to.

I think you just explained very well why we throw an error. Your =
arguments against
silent mode are valid and we don't need to consider RETURN_ERROR any =
further.

Current master behavior throws an error, in this thread author propose =
to backport it.
This might be behavior change for some users. But it seems to me we have =
to backport,
because

SELECT '42'::jsonb @? '$"no_such_var"';

should not return true. What do you think?


Best regards, Andrey Borodin.=