Hello Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12271 to look at the new patch set (#4). Change subject: IMPALA-7999: clean up start-*d.sh scripts ...................................................................... IMPALA-7999: clean up start-*d.sh scripts Delete these wrapper scripts and replace with a generic start-daemon.sh script that sets environment variables without the other logic. Move the logic for setting JAVA_TOOL_OPTIONS into start-impala-cluster.py. Port across the kerberized minicluster logic (which has probably bitrotted) in case it needs to be revived. Remove --verbose option that didn't appear to be useful (it claims to print daemon output to the console, but output is still redirected regardless). Removed a level of quoting in custom cluster test argument handling - this was made unnecessary by properly escaping arguments with pipes.escape() in run_daemon(). Testing: * TODO: complete exhaustive run * TODO: run on CentOS 6 to confirm we didn't reintroduce Popen issue worked around by kwho. Change-Id: Ib67444fd4def8da119db5d3a0832ef1de15b068b --- D bin/start-catalogd.sh A bin/start-daemon.sh M bin/start-impala-cluster.py D bin/start-impalad.sh D bin/start-statestored.sh M tests/common/custom_cluster_test_suite.py M tests/common/impala_cluster.py M tests/custom_cluster/test_breakpad.py M tests/custom_cluster/test_redaction.py M tests/custom_cluster/test_scratch_disk.py 10 files changed, 161 insertions(+), 337 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/12271/4 -- To view, visit http://gerrit.cloudera.org:8080/12271 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib67444fd4def8da119db5d3a0832ef1de15b068b Gerrit-Change-Number: 12271 Gerrit-PatchSet: 4 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>