NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit529ab7b

committed

Fix timing issue in new subscription truncate test

We need to wait for the initial sync of all subscriptions. Onsome (faster?) machines, this didn't make a difference, butthe (slower?) buildfarm machines are upset.

1 parentbf75fe4 commit529ab7bCopy full SHA for 529ab7b

File tree

1 file changed

-1

lines changed

src/test/subscription/t
- 010_truncate.pl

1 file changed

-1

lines changed

`‎src/test/subscription/t/010_truncate.pl`

Lines changed: 5 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,11 @@`
`61`	`61`	`$node_subscriber->safe_psql('postgres',`
`62`	`62`	`"CREATE SUBSCRIPTION sub3 CONNECTION '$publisher_connstr application_name=sub3' PUBLICATION pub3");`
`63`	`63`
`64`		`-$node_publisher->wait_for_catchup('sub1');`
	`64`	`+# Wait for initial sync of all subscriptions`
	`65`	`+my$synced_query =`
	`66`	`+"SELECT count(1) = 0 FROM pg_subscription_rel WHERE srsubstate NOT IN ('r', 's');";`
	`67`	`+$node_subscriber->poll_query_until('postgres',$synced_query)`
	`68`	`+ordie"Timed out while waiting for subscriber to synchronize data";`
`65`	`69`
`66`	`70`	`# insert data to truncate`
`67`	`71`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit529ab7b

File tree

1 file changed

1 file changed

`‎src/test/subscription/t/010_truncate.pl`

0 commit comments