aboutsummaryrefslogtreecommitdiff
path: root/src/backend/access/gist/gistget.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2021-09-17 15:41:16 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2021-09-17 15:41:16 -0400
commit4d5b4483db1c6370861ca968edd753ab4dc03f9d (patch)
treecab98b05676110f502d3bf11aa2cd0a94a999808 /src/backend/access/gist/gistget.c
parent388726753b638fb9938883bdd057b2ffe6f950f5 (diff)
downloadpostgresql-4d5b4483db1c6370861ca968edd753ab4dc03f9d.tar.gz
postgresql-4d5b4483db1c6370861ca968edd753ab4dc03f9d.zip
Fix pull_varnos to cope with translated PlaceHolderVars.
Commit 55dc86eca changed pull_varnos to use (if possible) the associated ph_eval_at for a PlaceHolderVar. I missed a fine point though: we might be looking at a PHV in the quals or tlist of a child appendrel, in which case we need to compute a ph_eval_at value that's been translated in the same way that the PHV itself has been (cf. adjust_appendrel_attrs). Fortunately, enough info is available in the PlaceHolderInfo to make such translation possible without additional outside data, so we don't need another round of uglification of planner APIs. This is a little bit complicated, but since it's a hard-to-hit corner case, I'm not much worried about adding cycles here. Per report from Jaime Casanova. Back-patch to v12, like the previous commit. Discussion: https://postgr.es/m/20210915230959.GB17635@ahch-to
Diffstat (limited to 'src/backend/access/gist/gistget.c')
0 files changed, 0 insertions, 0 deletions