Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/8310
Change subject: [client-stress-test] do not crash if client failed
......................................................................
[client-stress-test] do not crash if client failed
Do not crash the test if failed to build Kudu client instance in the
TestUniqueClientIds scenario, just use regular ASSERT_OK() instead,
allowing the test cluster to wind down, so it would be possible to
check on the status of the master and tablet servers during shutdown.
With crashing the test on an error, it's not possible to figure out
whether client <--> server negotiation failed due to a crashed server
or something else.
As for the context, the ClientStressTest.TestUniqueClientIds test
has been flaky for a while, failing with errors like the following:
Client connection negotiation failed: client connection to IP:port: \
Read zero bytes on a blocking Recv() call: Transferred 0 of 4 bytes
Change-Id: I558fcb02f697d1ec216538416826d96dcda32a1e
---
M src/kudu/integration-tests/client-stress-test.cc
1 file changed, 7 insertions(+), 6 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/10/8310/1
--
To view, visit http://gerrit.cloudera.org:8080/8310
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I558fcb02f697d1ec216538416826d96dcda32a1e
Gerrit-Change-Number: 8310
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
