Skip to content

Commit

Permalink
[#7889] [tests] Fix PgIndexBackfillTest.Large
Browse files Browse the repository at this point in the history
Summary: Fix assert to account for lowered rpc timeout (in 2fcae00)  which may result in more than one backfill RPC per tablet.

Test Plan: ybd asan --cxx-test pg_index_backfill-test --gtest_filter PgIndexBackfillTest.Large  -n 100 --tp 1

Reviewers: jason

Reviewed By: jason

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D12951
  • Loading branch information
amitanandaiyer committed Sep 13, 2021
1 parent a311619 commit 8f88f5f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/yb/yql/pgwrapper/pg_index_backfill-test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -376,7 +376,7 @@ TEST_F(PgIndexBackfillTest, YB_DISABLE_TEST_IN_TSAN(Large)) {
TestLargeBackfill(kNumRows);
int expected_calls = cluster_->num_tablet_servers() * kTabletsPerServer;
auto actual_calls = ASSERT_RESULT(TotalBackfillRpcCalls(cluster_.get()));
ASSERT_EQ(actual_calls, expected_calls);
ASSERT_GE(actual_calls, expected_calls);
}

class PgIndexBackfillTestChunking : public PgIndexBackfillTest {
Expand Down

0 comments on commit 8f88f5f

Please sign in to comment.