Skip to content

Commit 4fed60e

Browse files
authored
chore(stage7): merge t/335 pin-order fix (PR #36)
Primary pins moved before their start; restores the pre-merge failure shape (pre-existing ADG test-10 leg only). Spec: spec-7.3-lms-worker-pool.md
2 parents 2c428bf + 3f71404 commit 4fed60e

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/test/cluster_tap/t/335_adg_two_thread_rfs_apply.pl

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -243,6 +243,10 @@ sub wait_for_log_match_from
243243
$primary0->append_conf('postgresql.conf', "cluster.node_id = 0\n");
244244
$primary1->append_conf('postgresql.conf', $primary_common_conf);
245245
$primary1->append_conf('postgresql.conf', "cluster.node_id = 1\n");
246+
# spec-7.3: pin the pool BEFORE the primaries start (the block further down
247+
# runs after ->start and cannot help them) -- one data port per node here.
248+
$primary0->append_conf('postgresql.conf', "cluster.lms_workers = 1\n");
249+
$primary1->append_conf('postgresql.conf', "cluster.lms_workers = 1\n");
246250
write_pair_conf($primary0, $primary1, 'adg2_primary');
247251
$primary0->start;
248252
$primary1->start;
@@ -296,8 +300,6 @@ sub wait_for_log_match_from
296300
# cross-wires consecutive free ports (HELLO DATA worker mismatch). Pin the
297301
# pool to one worker: N=1 is the spec-7.2 topology identity this rig was
298302
# written against.
299-
$primary0->append_conf('postgresql.conf', "cluster.lms_workers = 1\n");
300-
$primary1->append_conf('postgresql.conf', "cluster.lms_workers = 1\n");
301303
$standby0->append_conf('postgresql.conf', "cluster.lms_workers = 1\n");
302304
$standby1->append_conf('postgresql.conf', "cluster.lms_workers = 1\n");
303305
write_pair_conf($standby0, $standby1, 'adg2_standby');

0 commit comments

Comments
 (0)