public inbox for [email protected]  

help / color / mirror / Atom feed

From: Richard Guo <[email protected]>
To: Sergey Shinderuk <[email protected]>
Cc: [email protected]
Cc: Tom Lane <[email protected]>
Cc: David Rowley <[email protected]>
Subject: Re: BUG #19412: Wrong query result with not null constraint
Date: Thu, 19 Feb 2026 10:49:44 +0900
Message-ID: <CAMbWs481y0EmS7fFJE9DEjjHGJO4mh9nPmusPDQ77Psxgo96ww@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<CAApHDvpXarz-D6tP-7y4ZrtnhXtdbRY0kQxRDQTVc3Z9DckYVw@mail.gmail.com>
	<CAMbWs49kVgb_wS4VFuQpfZ+Re4tWVn7vmheR6HqKppnNuEaH-g@mail.gmail.com>
	<CAMbWs49NVUNpQPwGb56SCSQL-bkjtxnkCb6Ot7dBxGndJoA3Pw@mail.gmail.com>
	<[email protected]>

On Wed, Feb 18, 2026 at 9:03 PM Sergey Shinderuk
<[email protected]> wrote:
> I'm not familiar with the code, just curios. There is a long comment
> above saying "You might think we need to adjust var->varnullingrels, but
> that shouldn't need any changes." Doesn't it need an update?

No, I don't think we need to update it.  That comment explains why
varnullingrels do not require translation (since they are outer join
relids, not baserel relids).  It's unrelated to what this patch does,
which is about propagating varnullingrels into the translated Var.

- Richard

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: BUG #19412: Wrong query result with not null constraint
  In-Reply-To: <CAMbWs481y0EmS7fFJE9DEjjHGJO4mh9nPmusPDQ77Psxgo96ww@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox