Re: [PATCH] SAOP nullability analysis below NOT/BooleanTest

From: Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Richard Guo <guofenglinux(at)gmail(dot)com>
Subject: Re: [PATCH] SAOP nullability analysis below NOT/BooleanTest
Date: 2026-07-28 17:06:33
Message-ID: CAJTYsWWPcHj_mhW2Mj5wUsZ3FC2sGh2aOBiU9afNRkP4RB9fOA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Tue, 28 Jul 2026 at 22:33, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com> writes:
> > While looking at the recent work around converting NOT IN sublinks to
> > anti-joins, I came across what looks like a possible issue in the
> > ScalarArrayOpExpr handling in find_nonnullable_rels() and
> > find_nonnullable_vars().
>
> This is an odd one. Why did I add the falseOK argument to
> is_strict_saop and then not use it in all three callers? Sadly, I don't
> recall what I was thinking twenty years ago. But you're clearly right
> that all of them should be accounting for this effect.
>
> I'm inclined to merge this patch with your other SAOP correction,
> since they're so closely related.
>

Sounds good to me, thanks for looking in!

Regards,
Ayush

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tristan Partin 2026-07-28 17:08:09 Re: Support UUIDv6 in uuid_extract_timestamp()
Previous Message Tom Lane 2026-07-28 17:03:15 Re: [PATCH] SAOP nullability analysis below NOT/BooleanTest