Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit f1ef111

Browse files
committedMar 10, 2025
Fix a few more redundant calls of GetLatestSnapshot()
Commit 2367503 fixed this in RelationFindReplTupleByIndex(), but I missed two other similar cases. Per report from Ranier Vilela. Discussion: https://www.postgresql.org/message-id/CAEudQArUT1dE45WN87F-Gb7XMy_hW6x1DFd3sqdhhxP-RMDa0Q@mail.gmail.com Backpatch-through: 13
1 parent c1dd3a9 commit f1ef111

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎src/backend/executor/execReplication.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ RelationFindReplTupleSeq(Relation rel, LockTupleMode lockmode,
434434

435435
PushActiveSnapshot(GetLatestSnapshot());
436436

437-
res = table_tuple_lock(rel, &(outslot->tts_tid), GetLatestSnapshot(),
437+
res = table_tuple_lock(rel, &(outslot->tts_tid), GetActiveSnapshot(),
438438
outslot,
439439
GetCurrentCommandId(false),
440440
lockmode,

0 commit comments

Comments
 (0)
Please sign in to comment.