Re: Assert failure in try_nestloop_path()

On Tue, Sep 8, 2026 at 11:45 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Ah. Do you want to just merge those patches and push them as one?
> I think we want HEAD-and-19-only for all these changes.

Yes. I've merged the clone-dedup fix and your
drop-redundant-equivalence-clauses fix into a single patch (0001
attached), removed the find_clause_by_serial cross-checks, and
un-commented the assertion in get_joinrel_parampathinfo, which now
holds with your fix in place.

I've kept the create_join_clause change as a separate patch (0002),
since it's a distinct issue. There the trouble is that two
RestrictInfos for the same condition end up with different
rinfo_serials, rather than that we enforce multiple clones of the
same clause.

I plan to push both to HEAD and v19 but no further if there are no
objections.

- Richard