Re: Pathify RHS unique-ification for semijoin planning

From: Richard Guo <guofenglinux(at)gmail(dot)com>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andy Fan <zhihuifan1213(at)163(dot)com>, wenhui qiu <qiuwenhuifx(at)gmail(dot)com>
Subject: Re: Pathify RHS unique-ification for semijoin planning
Date: 2025-07-16 02:30:35
Message-ID: CAMbWs48vr3VkUOmy-2_Skapk8RTHYuGngAhEQdGVyPO2PQ+x2g@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jul 4, 2025 at 10:41 AM Richard Guo <guofenglinux(at)gmail(dot)com> wrote:
> On Thu, Jul 3, 2025 at 7:06 PM Richard Guo <guofenglinux(at)gmail(dot)com> wrote:
> > This patch does not apply again, so here is a new rebase.
> >
> > This version also fixes an issue related to parameterized paths: if
> > the RHS has LATERAL references to the LHS, unique-ification becomes
> > meaningless because the RHS depends on the LHS, and such paths should
> > not be generated.

> (The cc list is somehow lost; re-ccing.)

The CI reports a test failure related to this patch, although I'm
quite confident it's unrelated to the changes introduced here.

The failure is: recovery/009_twophase time out (After 1000 seconds)

In any case, here's a freshly rebased version.

Hi Tom, I wonder if you've had a chance to look at this patch. It
would be great to have your input.

Thanks
Richard

Attachment Content-Type Size
v5-0001-Pathify-RHS-unique-ification-for-semijoin-plannin.patch application/octet-stream 106.4 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ajin Cherian 2025-07-16 03:08:04 Re: 024_add_drop_pub.pl might fail due to deadlock
Previous Message Richard Guo 2025-07-16 01:57:51 Re: Reduce "Var IS [NOT] NULL" quals during constant folding