Commitef585de

committed

Fix two-phase commit test for recovery mode

The original code had a race condition because it never ensured thestandby was caught up before proceeding; add a wait similar to everyother place that does this.Author: Michaël PaquierDiscussion:https://postgr.es/m/CAB7nPqTm9p+LCm1mVJYvgpwagRK+uibT-pKq0O2-paOWxT62jw@mail.gmail.com

1 parentf15b76a commitef585deCopy full SHA for ef585de

File tree

-0

lines changed

-0

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -331,6 +331,14 @@ sub configure_and_reload`
`331`	`331`	`CHECKPOINT;`
`332`	`332`	`COMMIT PREPARED 'xact_009_13';");`
`333`	`333`
	`334`	`+# Ensure that last transaction is replayed on standby.`
	`335`	`+my$cur_master_lsn =`
	`336`	`+$cur_master->safe_psql('postgres',"SELECT pg_current_wal_lsn()");`
	`337`	`+my$caughtup_query =`
	`338`	`+"SELECT '$cur_master_lsn'::pg_lsn <= pg_last_wal_replay_lsn()";`
	`339`	`+$cur_standby->poll_query_until('postgres',$caughtup_query)`
	`340`	`+ordie"Timed out while waiting for standby to catch up";`
	`341`	`+`
`334`	`342`	`$cur_standby->psql(`
`335`	`343`	`'postgres',`
`336`	`344`	`"SELECT count(*) FROM t_009_tbl2",`

Comments

(0)