]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Remove faulty Assert in partitioned INSERT...ON CONFLICT DO UPDATE.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 22 Jan 2026 23:35:31 +0000 (18:35 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 22 Jan 2026 23:35:31 +0000 (18:35 -0500)
commit3f56de3aad1bcffae01dc2c22a41518a18d25d77
treecb0a2cb3bb659fe3811d264d3791419e5c99a735
parent3f2ab3f34da9d4299255a972a1c022a152bbf3f7
Remove faulty Assert in partitioned INSERT...ON CONFLICT DO UPDATE.

Commit f16241bef mistakenly supposed that INSERT...ON CONFLICT DO
UPDATE rejects partitioned target tables.  (This may have been
accurate when the patch was written, but it was already obsolete
when committed.)  Hence, there's an assertion that we can't see
ItemPointerIndicatesMovedPartitions() in that path, but the assertion
is triggerable.

Some other places throw error if they see a moved-across-partitions
tuple, but there seems no need for that here, because if we just retry
then we get the same behavior as in the update-within-partition case,
as demonstrated by the new isolation test.  So fix by deleting the
faulty Assert.  (The fact that this is the fix doubtless explains
why we've heard no field complaints: the behavior of a non-assert
build is fine.)

The TM_Deleted case contains a cargo-culted copy of the same Assert,
which I also deleted to avoid confusion, although I believe that one
is actually not triggerable.

Per our code coverage report, neither the TM_Updated nor the
TM_Deleted case were reached at all by existing tests, so this
patch adds tests for both.

Reported-by: Dmitry Koval <d.koval@postgrespro.ru>
Author: Joseph Koshakow <koshy44@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/f5fffe4b-11b2-4557-a864-3587ff9b4c36@postgrespro.ru
Backpatch-through: 14
src/backend/executor/nodeModifyTable.c
src/test/isolation/expected/insert-conflict-do-update-4.out [new file with mode: 0644]
src/test/isolation/isolation_schedule
src/test/isolation/specs/insert-conflict-do-update-4.spec [new file with mode: 0644]