Fix missed step in removal of useless RESULT RTEs in the planner.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 5 Dec 2020 21:16:13 +0000 (16:16 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 5 Dec 2020 21:16:13 +0000 (16:16 -0500)
commite98c900993e89ad9278cdfbf0ba5495381a1faac
treef748596180c6f73079bcd14fc6cda783634128f8
parent36a4ac20fcf31361bd42b63b1b3390b28827a69e
Fix missed step in removal of useless RESULT RTEs in the planner.

Commit 4be058fe9 forgot that the append_rel_list would already be
populated at the time we remove useless result RTEs, and it might contain
PlaceHolderVars that need to be adjusted like the ones in the main parse
tree.  This could lead to "no relation entry for relid N" failures later
on, when the planner tries to do something with an unadjusted PHV.

Per report from Tom Ellis.  Back-patch to v12 where the bug came in.

Discussion: https://postgr.es/m/20201205173056.GF30712@cloudinit-builder
src/backend/optimizer/prep/prepjointree.c
src/test/regress/expected/join.out
src/test/regress/sql/join.sql