On Sat, Sep 13, 2025 at 2:32 AM Robert Haas <robertmh...@gmail.com> wrote: > We sort of got started down this path because, reviewing v4-0003, > Richard commented that I might be able to sanity-check > something-or-other about RTE_JOIN RTIs instead of just focusing on > baserels. From there, this sub-thread has turned into a discussion of > exactly what that sanity check should be.
Yeah, when commenting on v4-0003 about including outer join relids in the assertion, I had a feeling that it might take the discussion off topic -- sorry that it did. That's why I suggested moving this part of discussion to a separate thread. I still think that cross-checking outer join relids isn't a requirement for committing your patches, so I'm totally fine if you end up with a version that doesn't assert outer join relids. - Richard