Todd Lipcon has submitted this change and it was merged.
Change subject: [tests] improvements on client_samples test
......................................................................
[tests] improvements on client_samples test
This patch adds the following improvements for the client_samples-test:
* Added consistency check to make sure the client sample application
talks to the dedicated master and tablet servers. In the prior
version of the script, if running at the machine where Kudu master
and tablet server are running at default ports bound to all network
interfaces (i.e. 0.0.0.0-wildcard bound), the test inadvertently
used that Kudu cluster, not the dedicated one.
* Increased the timeout for the master and the tablet server to start
listening on their RPC ports up to 30 seconds. It was observed that
on busy test VMs it takes a long time for the master and the tablet
server to bootstrap and start serving client requests.
* Updated the setting of the shell options to make them work even if
running the script via explicit interpreter, like
/bin/bash client_samples-tesh.sh
Change-Id: I5cbaa01313cdf49117bbf8e70e7a22a35fe47021
Reviewed-on: http://gerrit.cloudera.org:8080/7359
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Todd Lipcon <[email protected]>
---
M src/kudu/client/client_samples-test.sh
1 file changed, 69 insertions(+), 14 deletions(-)
Approvals:
Todd Lipcon: Looks good to me, approved
Alexey Serbin: Verified
--
To view, visit http://gerrit.cloudera.org:8080/7359
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5cbaa01313cdf49117bbf8e70e7a22a35fe47021
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
