This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
The following commit(s) were added to refs/heads/master by this push:
new b60b6fd77 [client-test] more robust MinReplicationFactor scenario
b60b6fd77 is described below
commit b60b6fd7751b3b67156f460fb46dabd2db00d9d7
Author: Alexey Serbin <[email protected]>
AuthorDate: Mon Jul 11 19:01:32 2022 -0700
[client-test] more robust MinReplicationFactor scenario
This patch updates the ReplicationFactorLimitsTest.MinReplicationFactor
test scenario to make it more robust. With this update, test failures
like below shouldn't happen anymore:
src/kudu/client/client-test.cc:8722: Failure
Value of: s.ToString()
Expected: has substring "not enough live tablet servers to create a table
with the requested replication factor 3; 1 tablet servers are alive"
Actual: "Invalid argument: Error creating table foobar on the master:
not enough live tablet servers to create a table with the requested replication
factor 3; 0 tablet servers are alive"
Change-Id: I509f04067f28be566f655050f1b29f38654df24b
Reviewed-on: http://gerrit.cloudera.org:8080/18719
Reviewed-by: Mahesh Reddy <[email protected]>
Tested-by: Kudu Jenkins
Reviewed-by: Yingchun Lai <[email protected]>
---
src/kudu/client/client-test.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/kudu/client/client-test.cc b/src/kudu/client/client-test.cc
index 25c0961e6..97a38618c 100644
--- a/src/kudu/client/client-test.cc
+++ b/src/kudu/client/client-test.cc
@@ -8695,7 +8695,7 @@ TEST_F(ReplicationFactorLimitsTest, MinReplicationFactor)
{
const vector<pair<int, string>> cases = {
{2, "illegal replication factor 2: replication factor must be odd"},
{3, "not enough live tablet servers to create a table with the requested
"
- "replication factor 3; 1 tablet servers are alive"},
+ "replication factor 3"},
};
for (auto i = 1; i < cluster_->num_tablet_servers(); ++i) {
