Fix handling of self-modified tuples in MERGE.
authorDean Rasheed
Thu, 7 Mar 2024 09:55:39 +0000 (09:55 +0000)
committerDean Rasheed
Thu, 7 Mar 2024 09:55:39 +0000 (09:55 +0000)
commitdd73d10adf0048d2d8dc4e94cd64ce5e5ff45a4b
treea77c8d49968f7660aa482f15b42d78405ff716b3
parentc46817ee512afa156561cce2dd6dc8440fe98d55
Fix handling of self-modified tuples in MERGE.

When an UPDATE or DELETE action in MERGE returns TM_SelfModified,
there are 2 possible causes:

1). The target tuple was already updated or deleted by the current
    command. This can happen if the target row joins to more than one
    source row, and the SQL standard explicitly says that this must be
    an error.

2). The target tuple was already updated or deleted by a later command
    in the current transaction. This can happen if the tuple is
    modified by a BEFORE trigger or a volatile function used in the
    query, and should be an error for the same reason that it is in a
    plain UPDATE or DELETE command.

In MERGE's primary error handling block, it failed to check for (2),
causing it to return a misleading error message in such cases.

In the secondary error handling block, following a concurrent update
from another session, it failed to check for (1), causing it to
silently ignore target rows joined to more than one source row,
instead of reporting an error.

Fix this, and add tests for both of these cases.

Per report from Wenjiang Zhang. Back-patch to v15, where MERGE was
introduced.

Discussion: https://postgr.es/m/tencent_41DE0FF443FE14B94A5898D373792109E408%40qq.com
src/backend/executor/nodeModifyTable.c
src/test/isolation/expected/merge-update.out
src/test/isolation/specs/merge-update.spec
src/test/regress/expected/triggers.out
src/test/regress/sql/triggers.sql