Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/8459
Change subject: Fix errant, newline-including log directory.
......................................................................
Fix errant, newline-including log directory.
We've seen cases where a directory named "cluster\n " sneaks
into the logs directory. The intention is that $IMPALA_ALL_LOGS_DIRS
is a space-separated list, but clean.sh (as opposed to buildall.sh)
treats it as a single argument.
I tested this manually:
Before:
$ls logs/
be_tests/ custom_cluster_tests/ data_loading/ ee_tests/ fe_tests/
Bad command:
$mkdir -p "${IMPALA_ALL_LOGS_DIRS}"
After:
$ls logs/
be_tests/ cluster? / custom_cluster_tests/ data_loading/ ee_tests/
fe_tests/
Change-Id: Id18404c3f3fc0008f67b01afbac9d908532fc317
---
M bin/clean.sh
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/59/8459/1
--
To view, visit http://gerrit.cloudera.org:8080/8459
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id18404c3f3fc0008f67b01afbac9d908532fc317
Gerrit-Change-Number: 8459
Gerrit-PatchSet: 1
Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
