Fix miscomputation of direct_lateral_relids for join relations.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 30 Nov 2020 17:22:43 +0000 (12:22 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 30 Nov 2020 17:22:43 +0000 (12:22 -0500)
commit666a4de939bf78df7295c35899a5f2e89eaea382
tree8703c86f722a219fc90a58a9482e3fcd474da0fa
parent74d6fb0a037a7453693418e1069718a7f08ba31e
Fix miscomputation of direct_lateral_relids for join relations.

If a PlaceHolderVar is to be evaluated at a join relation, but
its value is only needed there and not at higher levels, we neglected
to update the joinrel's direct_lateral_relids to include the PHV's
source rel.  This causes problems because join_is_legal() then won't
allow joining the joinrel to the PHV's source rel at all, leading
to "failed to build any N-way joins" planner failures.

Per report from Andreas Seltenreich.  Back-patch to 9.5
where the problem originated.

Discussion: https://postgr.es/m/87blfgqa4t.fsf@aurora.ydns.eu
src/backend/optimizer/util/placeholder.c
src/test/regress/expected/join.out
src/test/regress/sql/join.sql