postgres.git / summary / log / commit / refs

commit    aab422af95c395e01436f9fc0cb74ff96550143e
Author:   Fujii Masao <fujii@postgresql.org>
Date:     Fri Apr 04 04:32:46 2025 +0000
First released in: 18.0, 13.21, 14.18, 15.13, 16.9, 17.5

    Fix logical decoding test to correctly check slot removal on standby.
    
    The regression test for logical decoding verifies whether a logical slot
    is correctly dropped on a standby when its associated database is dropped.
    However, the test mistakenly retrieved slot information from the primary
    instead of the standby, causing incorrect behavior.
    
    This commit fixes the issue by ensuring the test correctly checks the slot
    on the standby.
    
    Back-patch to all supported versions.
    
    Author: Hayato Kuroda <kuroda.hayato@fujitsu.com>
    Reviewed-by: Fujii Masao <masao.fujii@gmail.com>
    Discussion: https://postgr.es/m/1fdfd020-a509-403c-bd8f-a04664aba148@oss.nttdata.com
    Backpatch-through: 13


 src/test/recovery/t/010_logical_decoding_timelines.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/test/recovery/t/010_logical_decoding_timelines.pl b/src/test/recovery/t/010_logical_decoding_timelines.pl
index 7685b98589b..40f4fb0d918 100644
--- a/src/test/recovery/t/010_logical_decoding_timelines.pl
+++ b/src/test/recovery/t/010_logical_decoding_timelines.pl
@@ -94,7 +94,7 @@ is( $node_replica->safe_psql(
 		'postgres', q[SELECT 1 FROM pg_database WHERE datname = 'dropme']),
 	'',
 	'dropped DB dropme on standby');
-is($node_primary->slot('dropme_slot')->{'plugin'},
+is($node_replica->slot('dropme_slot')->{'plugin'},
 	'', 'logical slot was actually dropped on standby');
 
 # Back to testing failover...


[parent: 5570e103bf84]