Re: Assert failure in try_nestloop_path()

From: Richard Guo <guofenglinux(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Assert failure in try_nestloop_path()
Date: 2026-09-04 01:50:18
Message-ID: CAMbWs4-2Wu7dQLuaacH0+vnsUGGZb-76upLFTPCGLX7a6bU1_w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Sep 3, 2026 at 11:57 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Richard Guo <guofenglinux(at)gmail(dot)com> writes:
> > Attached is a patch that teaches join_is_legal() to reject a join
> > whose minimum parameterization includes an outer-join relid, if that
> > outer join cannot be formed outside the join.

> This looks like a good fix to me. Interesting that we've failed
> to notice this bug up to now.

Thanks for looking! I think part of the reason we've failed to notice
this bug is that such joins used to be rejected by have_dangerous_phv(),
which was only removed in v18

I've pushed this patch and back-patched it to v18.

- Richard

In response to

Browse pgsql-hackers by date

  From Date Subject
Previous Message Michael Paquier 2026-09-04 01:35:25 Re: Remove fcinfo from statistics update internal functions