Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env ......................................................................
IMPALA-3229: Don't assume that AUX exists just because of shell env This caused breakage of the custom cluster tests - bin/impala-config.sh sets IMPALA_AUX_TEST_HOME to $IMPALA_HOME/../Impala-auxiliary-tests, unless it is already set, but this directory is not guaranteed to exist. bin/run-custom-cluster-tests.sh was expecting that if IMPALA_AUX_TEST_HOME is set, then that directory must exist. While I'm changing bin/run-custom-cluster-tests.sh, add some quotes to protect against paths with spaces in them. Change-Id: Ieffb6d52c5f95d3257de8c47a039ccb0d45f0867 Reviewed-on: http://gerrit.cloudera.org:8080/4563 Reviewed-by: Jim Apple <[email protected]> Tested-by: Internal Jenkins --- M tests/run-custom-cluster-tests.sh 1 file changed, 16 insertions(+), 13 deletions(-) Approvals: Jim Apple: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4563 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieffb6d52c5f95d3257de8c47a039ccb0d45f0867 Gerrit-PatchSet: 6 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]>
