]> granicus.if.org Git - postgresql/commit
Use PlaceHolderVars within the quals of a FULL JOIN.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 14 Oct 2018 17:07:29 +0000 (13:07 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 14 Oct 2018 17:07:29 +0000 (13:07 -0400)
commit7d4a10e260083bc571ea05592884084eabcc7afa
tree132bec3d0ce4e8e6bbd0c58811e1291effb006aa
parente9f42d529f990f94e1b7bdcec4a1111465c85326
Use PlaceHolderVars within the quals of a FULL JOIN.

This prevents failures in cases where we pull up a constant or var-free
expression from a subquery and put it into a full join's qual.  That can
result in not recognizing the qual as containing a mergejoin-able or
hashjoin-able condition.  A PHV prevents the problem because it is still
recognized as belonging to the side of the join the subquery is in.

I'm not very sure about the net effect of this change on plan quality.
In "typical" cases where the join keys are Vars, nothing changes.
In an affected case, the PHV-wrapped expression is less likely to be seen
as equal to PHV-less instances below the join, but more likely to be seen
as equal to similar expressions above the join, so it may end up being a
wash.  In the one existing case where there's any visible change in a
regression-test plan, it amounts to referencing a lower computation of a
COALESCE result instead of recomputing it, which seems like a win.

Given my uncertainty about that and the lack of field complaints,
no back-patch, even though this is a very ancient problem.

Discussion: https://postgr.es/m/32090.1539378124@sss.pgh.pa.us
src/backend/optimizer/prep/prepjointree.c
src/test/regress/expected/join.out
src/test/regress/sql/join.sql