database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-25 21:42:33 +03:00
Code

Fix query in new test to check tables are synced

Browse Source 
Rather than looking for tablesync workers, it is more reliable to see
the sync state of the tables.

Per note from Amit Kapila.
Discussion: https://postgr.es/m/CAA4eK1JSSD7FVwq+_rOme86jUZTQFzjsNU06hQ4-LiRt1xFmSg@mail.gmail.com
This commit is contained in:
Alvaro Herrera 2020-10-15 09:48:36 -03:00
parent 86dba33217
commit 7f47088183
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/test/subscription/t/100_bugs.pl
View File

@ -141,13 +141,13 @@ $node_twoways->safe_psql(
# We cannot rely solely on wait_for_catchup() here; it isn't sufficient # We cannot rely solely on wait_for_catchup() here; it isn't sufficient
# when tablesync workers might still be running. So in addition to that, # when tablesync workers might still be running. So in addition to that,
# we verify that no tablesync workers appear for the subscription. # verify that tables are synced.
# XXX maybe this should be integrated in wait_for_catchup() itself. # XXX maybe this should be integrated in wait_for_catchup() itself.
$node_twoways->wait_for_catchup('testsub'); $node_twoways->wait_for_catchup('testsub');
$node_twoways->poll_query_until( my $synced_query =
'd2', "SELECT count(1) = 0 FROM pg_subscription_rel WHERE srsubstate NOT IN ('r', 's');";
"SELECT count(*) FROM pg_stat_subscription WHERE subname = 'testsub' AND relid <> 0", $node_twoways->poll_query_until('d2', $synced_query)
"0"); or die "Timed out while waiting for subscriber to synchronize data";
is($node_twoways->safe_psql('d2', "SELECT count(f) FROM t"), is($node_twoways->safe_psql('d2', "SELECT count(f) FROM t"),
$rows * 2, "2x$rows rows in t"); $rows * 2, "2x$rows rows in t");