Re: Clean up remove_rel_from_query() after self-join elimination commit

On Tue, Apr 7, 2026 at 6:57 PM wenhui qiu <qiuwenhuifx(at)gmail(dot)com> wrote:
> Assert(!is_outer_join || joinrelids != NULL);

Worth asserting. If a caller sets sjinfo but passes NULL for
joinrelids, this would silently over-delete PHVs.

> Assert(!is_self_join || joinrelids == NULL);

I prefer to not add this one. It's not defending any invariant.

- Richard