diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-09-17 15:41:16 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-09-17 15:41:16 -0400 |
commit | e0b0d1eab48c85f961a39c263d2074ad11931920 (patch) | |
tree | d30e19d330db9162b1d6bc39fae15e22d55a9b97 /contrib/postgres_fdw/postgres_fdw.h | |
parent | 8767a86fa2b1d24537cca318a26fc68d7e457911 (diff) | |
download | postgresql-e0b0d1eab48c85f961a39c263d2074ad11931920.tar.gz postgresql-e0b0d1eab48c85f961a39c263d2074ad11931920.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 'contrib/postgres_fdw/postgres_fdw.h')
0 files changed, 0 insertions, 0 deletions