Richard Guo <guofengli...@gmail.com> writes: > Note that the evaluation of expression 'b.j + 1' now occurs below the > outer join. Is this something we need to be concerned about?
It seems more formally correct to me, but perhaps somebody would complain about possibly-useless expression evals. We could likely re-complicate the logic that inserts PHVs during pullup so that it looks for Vars it can apply the nullingrels to. Maybe there's an opportunity to share code with flatten_join_alias_vars? regards, tom lane