pgsql: Fix copying data into slots with FDW batching

From: Tomas Vondra <tomas(dot)vondra(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix copying data into slots with FDW batching
Date: 2021-06-16 21:54:40
Message-ID: E1ltdUe-0003Ei-Ns@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix copying data into slots with FDW batching

Commit b676ac443b optimized handling of tuple slots with bulk inserts
into foreign tables, so that the slots are initialized only once and
reused for all batches. The data was however copied into the slots only
after the initialization, inserting duplicate values when the slot gets
reused. Fixed by moving the ExecCopySlot outside the init branch.

The existing postgres_fdw tests failed to catch this due to inserting
data into foreign tables without unique indexes, and then checking only
the number of inserted rows. This adds a new test with both a unique
index and a check of inserted values.

Reported-by: Alexander Pyhalov
Discussion: https://postgr.es/m/7a8cf8d56b3d18e5c0bccd6cd42d04ac%40postgrespro.ru

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/99cea49d6525e30bc3768e4ffb95377e7584dea1

Modified Files
--------------
contrib/postgres_fdw/expected/postgres_fdw.out | 82 +++++++++++++++++++++++++-
contrib/postgres_fdw/sql/postgres_fdw.sql | 29 ++++++++-
src/backend/executor/nodeModifyTable.c | 10 ++--
3 files changed, 115 insertions(+), 6 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2021-06-16 23:30:34 pgsql: Fix plancache refcount leak after error in ExecuteQuery.
Previous Message Tom Lane 2021-06-16 15:52:13 pgsql: Improve SQLSTATE reporting in some replication-related code.