pgsql: Fix postgres_fdw failure with whole-row Vars of type RECORD. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix postgres_fdw failure with whole-row Vars of type RECORD.
Date
Msg-id E1lpJqM-0004AH-Vw@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix postgres_fdw failure with whole-row Vars of type RECORD.

Commit 86dc90056 expects that FDWs can cope with whole-row Vars for
their tables, even if the Vars are marked with vartype RECORDOID.
Previously, whole-row Vars generated by the planner had vartype equal
to the relevant table's rowtype OID.  (The point behind this change is
to enable sharing of resjunk columns across inheritance child tables.)

It turns out that postgres_fdw fails to cope with this, though through
bad fortune none of its test cases exposed that.  Things mostly work,
but when we try to read back a value of such a Var, the expected
rowtype is not available to record_in().  Fortunately, it's not
difficult to hack up the tupdesc that controls this process to
substitute the foreign table's rowtype for RECORDOID.  Thus we can
solve the runtime problem while still sharing the resjunk column
with other tables.

Per report from Alexander Pyhalov.

Discussion: https://postgr.es/m/7817fb9ebd6661cdf9b67dec6e129a78@postgrespro.ru

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f61db909dfb94f3411f8719916601a11a905b95e

Modified Files
--------------
contrib/postgres_fdw/expected/postgres_fdw.out | 25 ++++++++++++
contrib/postgres_fdw/postgres_fdw.c            | 55 +++++++++++++++++++++++++-
contrib/postgres_fdw/sql/postgres_fdw.sql      |  8 ++++
3 files changed, 86 insertions(+), 2 deletions(-)


pgsql-committers by date:

Previous
From: David Rowley
Date:
Subject: pgsql: Doc: Fix some spelling mistakes
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Fix subtransaction test for Python 3.10