On Tue, Sep 8, 2026 at 11:08 PM Tom Lane <[email protected]> wrote: > I think the find_clause_by_serial tests are just clutter, really.
Yeah, we can drop it, along with the two Assert(equal(...)) checks that used it. I'd added it only to increase my confidence that I understood the situation correctly. > Also, why is the last code hunk in get_joinrel_parampathinfo > commented out? Because without your drop-redundant-equivalence-clauses patch the clause-recovery pass for dropped ECs re-adds a clause and trips that Assert (the cardinal_number query). It needs to be un-commented once your fix gets applied. - Richard
