pgsql: Fix creation of resjunk tlist entries for inherited mixedUPDATE - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix creation of resjunk tlist entries for inherited mixedUPDATE
Date
Msg-id E1eJSIM-0007SD-Sd@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix creation of resjunk tlist entries for inherited mixed UPDATE/DELETE.

rewriteTargetListUD's processing is dependent on the relkind of the query's
target table.  That was fine at the time it was made to act that way, even
for queries on inheritance trees, because all tables in an inheritance tree
would necessarily be plain tables.  However, the 9.5 feature addition
allowing some members of an inheritance tree to be foreign tables broke the
assumption that rewriteTargetListUD's output tlist could be applied to all
child tables with nothing more than column-number mapping.  This led to
visible failures if foreign child tables had row-level triggers, and would
also break in cases where child tables belonged to FDWs that used methods
other than CTID for row identification.

To fix, delay running rewriteTargetListUD until after the planner has
expanded inheritance, so that it is applied separately to the (already
mapped) tlist for each child table.  We can conveniently call it from
preprocess_targetlist.  Refactor associated code slightly to avoid the
need to heap_open the target relation multiple times during
preprocess_targetlist.  (The APIs remain a bit ugly, particularly around
the point of which steps scribble on parse->targetList and which don't.
But avoiding such scribbling would require a change in FDW callback APIs,
which is more pain than it's worth.)

Also fix ExecModifyTable to ensure that "tupleid" is reset to NULL when
we transition from rows providing a CTID to rows that don't.  (That's
really an independent bug, but it manifests in much the same cases.)

Add a regression test checking one manifestation of this problem, which
was that row-level triggers on a foreign child table did not work right.

Back-patch to 9.5 where the problem was introduced.

Etsuro Fujita, reviewed by Ildus Kurbangaliev and Ashutosh Bapat

Discussion: https://postgr.es/m/20170514150525.0346ba72@postgrespro.ru

Branch
------
REL9_5_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/39f180fdd1e68bf6a897c6ab8733014e74f56973

Modified Files
--------------
contrib/postgres_fdw/expected/postgres_fdw.out |  65 ++++++++++++++++
contrib/postgres_fdw/sql/postgres_fdw.sql      |  18 +++++
doc/src/sgml/fdwhandler.sgml                   |   7 +-
doc/src/sgml/rules.sgml                        |   8 +-
src/backend/executor/nodeModifyTable.c         |   3 +-
src/backend/optimizer/plan/planner.c           |  16 +---
src/backend/optimizer/prep/preptlist.c         | 102 +++++++++++++++++--------
src/backend/rewrite/rewriteHandler.c           |  74 ++++++++----------
src/include/optimizer/prep.h                   |   5 +-
src/include/rewrite/rewriteHandler.h           |   3 +
10 files changed, 202 insertions(+), 99 deletions(-)


pgsql-committers by date:

Previous
From: Simon Riggs
Date:
Subject: pgsql: Additional docs for toast_tuple_target changes
Next
From: Tom Lane
Date:
Subject: pgsql: Fix assorted syscache lookup sloppiness in partition-relatedcod