]> granicus.if.org Git - postgresql/commit
Fix handling of PlaceHolderVars in nestloop parameter management.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Nov 2011 04:50:58 +0000 (00:50 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Nov 2011 04:50:58 +0000 (00:50 -0400)
commit7e3bf99baa18524de6ef1492cb3057314da97e68
tree3ebe96ce2088e06cf2e2ee471ff5e259926cba46
parent1a77f8b63d159b88ceb6245fcb5e81a7f9ac9a22
Fix handling of PlaceHolderVars in nestloop parameter management.

If we use a PlaceHolderVar from the outer relation in an inner indexscan,
we need to reference the PlaceHolderVar as such as the value to be passed
in from the outer relation.  The previous code effectively tried to
reconstruct the PHV from its component expression, which doesn't work since
(a) the Vars therein aren't necessarily bubbled up far enough, and (b) it
would be the wrong semantics anyway because of the possibility that the PHV
is supposed to have gone to null at some point before the current join.
Point (a) led to "variable not found in subplan target list" planner
errors, but point (b) would have led to silently wrong answers.
Per report from Roger Niederland.
src/backend/executor/nodeNestloop.c
src/backend/optimizer/plan/createplan.c
src/backend/optimizer/plan/setrefs.c
src/backend/optimizer/plan/subselect.c
src/include/nodes/plannodes.h
src/include/nodes/relation.h
src/include/optimizer/subselect.h
src/test/regress/expected/join.out
src/test/regress/sql/join.sql