Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1313. Fix client_samples-test to allow parallel runs on the same machine ......................................................................
KUDU-1313. Fix client_samples-test to allow parallel runs on the same machine This switches the shell script to use the same pid-based trickery to generate a unique localhost IP address that the other tests use. Tested manually by running 'ctest -R samples &' four times in quick succession. Without the patch, 3/4 of the runs failed. With the patch, they all succeeded. Change-Id: Iae90683a1c2264501a5740848b524841500e261e Reviewed-on: http://gerrit.cloudera.org:8080/1919 Reviewed-by: Jean-Daniel Cryans Tested-by: Kudu Jenkins --- M src/kudu/client/client_samples-test.sh M src/kudu/client/samples/sample.cc 2 files changed, 26 insertions(+), 4 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/1919 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae90683a1c2264501a5740848b524841500e261e Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
