Fix bogus ctid requirement for dummy-root partitioned targets ExecInitModifyTable() unconditionally required a ctid junk column even when the target was a partitioned table. This led to spurious "could not find junk ctid column" errors when all children were excluded and only the dummy root result relation remained.
A partitioned table only appears in the result relations list when all leaf partitions have been pruned, leaving the dummy root as the sole entry. Assert this invariant (nrels == 1) and skip the ctid requirement. Also adjust ExecModifyTable() to tolerate invalid ri_RowIdAttNo for partitioned tables, which is safe since no rows will be processed in this case. Bug: #19099 Reported-by: Alexander Lakhin <[email protected]> Author: Amit Langote <[email protected]> Reviewed-by: Tender Wang <[email protected]> Reviewed-by: Kirill Reshke <[email protected]> Reviewed-by: Tom Lane <[email protected]> Discussion: https://postgr.es/m/19099-e05dcfa022fe553d%40postgresql.org Backpatch-through: 14 Branch ------ REL_14_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/6d2fa44d2a612ab73f537ecf5428e527c9223da9 Modified Files -------------- contrib/file_fdw/input/file_fdw.source | 14 ++++++++++++++ contrib/file_fdw/output/file_fdw.source | 11 +++++++++++ src/backend/executor/nodeModifyTable.c | 19 ++++++++++++++++--- 3 files changed, 41 insertions(+), 3 deletions(-)
