| 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
| 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 |