Re: Assert failure in try_nestloop_path() - Mailing list pgsql-hackers

From Richard Guo
Subject Re: Assert failure in try_nestloop_path()
Date
Msg-id CAMbWs4-2Wu7dQLuaacH0+vnsUGGZb-76upLFTPCGLX7a6bU1_w@mail.gmail.com
Whole thread
Responses Re: allow spread checkpoints when changing checksums online
Re: Introducing find_all_inheritors_ordered()
List pgsql-hackers
On Thu, Sep 3, 2026 at 11:57 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Richard Guo <guofenglinux@gmail.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



Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Rename of varatt_external to varatt_external_oid
Next
From: Tom Lane
Date:
Subject: Re: [RFC PATCH] Cost-based delayed projection for ORDER BY ... LIMIT