Yingchun Lai has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18719 )

Change subject: [client-test] more robust MinReplicationFactor scenario
......................................................................

[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]>
---
M src/kudu/client/client-test.cc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mahesh Reddy: Looks good to me, but someone else must approve
  Kudu Jenkins: Verified
  Yingchun Lai: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/18719
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I509f04067f28be566f655050f1b29f38654df24b
Gerrit-Change-Number: 18719
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Khazar Mammadli <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>

Reply via email to