Remove assertion for replication origins in PREPARE TRANSACTION
authorMichael Paquier <michael@paquier.xyz>
Tue, 14 Dec 2021 01:58:29 +0000 (10:58 +0900)
committerMichael Paquier <michael@paquier.xyz>
Tue, 14 Dec 2021 01:58:29 +0000 (10:58 +0900)
commit3f710fc2b48d6326958260f10bd139a1e87fcf53
tree6057b9b6572dcb45f922510207a9412b53d04a82
parente459dcfde80402ab55e2fe84c5be4483ce34fce1
Remove assertion for replication origins in PREPARE TRANSACTION

When using replication origins, pg_replication_origin_xact_setup() is an
optional choice to be able to set a LSN and a timestamp to mark the
origin, which would be additionally added to WAL for transaction commits
or aborts (including 2PC transactions).  An assertion in the code path
of PREPARE TRANSACTION assumed that this data should always be set, so
it would trigger when using replication origins without setting up an
origin LSN.  Some tests are added to cover more this kind of scenario.

Oversight in commit 1eb6d65.

Per discussion with Amit Kapila and Masahiko Sawada.

Discussion: https://postgr.es/m/YbbBfNSvMm5nIINV@paquier.xyz
Backpatch-through: 11
contrib/test_decoding/expected/replorigin.out
contrib/test_decoding/sql/replorigin.sql
src/backend/access/transam/twophase.c