Re: Reduce "Var IS [NOT] NULL" quals during constant folding

From: Richard Guo <guofenglinux(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: Tomas Vondra <tomas(at)vondra(dot)me>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Eisentraut <peter(at)eisentraut(dot)org>, David Rowley <dgrowleyml(at)gmail(dot)com>, Tender Wang <tndrwang(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Reduce "Var IS [NOT] NULL" quals during constant folding
Date: 2025-08-21 01:07:33
Message-ID: CAMbWs487wxHq0fUu+Cew7Y+n+wpY_B--PT-61syXrE40uZqErw@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Aug 20, 2025 at 11:11 PM Nathan Bossart
<nathandbossart(at)gmail(dot)com> wrote:
> On Wed, Aug 20, 2025 at 10:29:03AM +0900, Richard Guo wrote:
> > On Wed, Aug 20, 2025 at 2:38 AM Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
> >> There is still an open item for this one, but it's not clear whether we are
> >> planning to do anything about this for v18, especially since nobody has
> >> shown measurable performance impact. Does anyone want to argue for
> >> addressing this for v18, or shall we close the open item as "Won't Fix"?

> > I don't think we're likely to do anything about this for v18.
> > Actually, I still doubt that the extra table_open call brings any
> > measurable performance impact, especially since the lock is already
> > held and the relation is likely already present in the relcache.
> >
> > Also, I still don't think moving the expansion of virtual generated
> > columns to the rewriter (as Tom proposed) is a better idea. It turned
> > out to have several problems that need to be fixed with the help of
> > PHVs, which is why we moved the expansion into the planner.

> Okay. I have marked the v18 open item as "Won't Fix".

Thank you for helping with this.

Thanks
Richard

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Smith 2025-08-21 01:10:24 Re: Remove redundant assignment of a variable in function AlterPublicationTables
Previous Message Chao Li 2025-08-21 00:41:36 Re: Remove redundant assignment of a variable in function AlterPublicationTables