postgres.git / summary / log / commit / refs

commit    cb2053dbde7a7e577484f92a60c9bcfe1147472b
Author:   Álvaro Herrera <alvherre@kurilemu.de>
Date:     Tue Jul 21 15:18:02 2026 +0000

    Remove assertion added by commit 7dcea51c2a4d
    
    We've got no reports of problems.  Get rid of it.
    
    Author: Álvaro Herrera <alvherre@kurilemu.de>
    Backpatch-through: 19
    Discussion: https://postgr.es/m/alewd1f2G0kKeM1i@alvherre.pgsql


 src/backend/replication/logical/logical.c | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/src/backend/replication/logical/logical.c b/src/backend/replication/logical/logical.c
index 3541fc793e4..c30d40a8641 100644
--- a/src/backend/replication/logical/logical.c
+++ b/src/backend/replication/logical/logical.c
@@ -2103,7 +2103,6 @@ LogicalSlotAdvanceAndCheckSnapState(XLogRecPtr moveto,
 									bool *found_consistent_snapshot)
 {
 	LogicalDecodingContext *ctx;
-	ResourceOwner old_resowner PG_USED_FOR_ASSERTS_ONLY = CurrentResourceOwner;
 	XLogRecPtr	retlsn;
 
 	Assert(XLogRecPtrIsValid(moveto));
@@ -2162,18 +2161,8 @@ LogicalSlotAdvanceAndCheckSnapState(XLogRecPtr moveto,
 			 * might still have critical updates to do.
 			 */
 			if (record)
-			{
 				LogicalDecodingProcessRecord(ctx, ctx->reader);
 
-				/*
-				 * We used to have bugs where logical decoding would fail to
-				 * preserve the resource owner.  That's important here, so
-				 * verify that that doesn't happen anymore.  XXX this could be
-				 * removed once it's been battle-tested.
-				 */
-				Assert(CurrentResourceOwner == old_resowner);
-			}
-
 			CHECK_FOR_INTERRUPTS();
 		}
 


[parent: 02597b42e5ff]