]> 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)
commit9f7c803c91584fb6e4b45dc87de44bac370477a9
tree5d8e782e9a1ee652490bd224b67c417edb869fce
parenta3bbd60b94cfbc2fa91f0cc5b94c2105b176b303
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]