Michael Brown has posted comments on this change. Change subject: IMPALA-3329: custom cluster tests: set -max_log_files to unlimited ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2759/1/bin/start-impala-cluster.py File bin/start-impala-cluster.py: Line 28: DEFAULT_IMPALA_MAX_LOG_FILES = os.environ.get('IMPALA_MAX_LOG_FILES', 10) > This new env variable seems oddly specific. We already have the ability to Given that the ultimate goal is to get ALL logs from Jenkins, I want to export IMPALA_MAX_LOG_FILES=0 for the Jenkins builds in the preamble. That should prevent log rotation for the duration of the build. Implementing that this way seemed easiest. Moreover, adding a special option to do the same thing for statestore, catalog, and impala seemed easier than explicitly listing out the same argument for each daemon. It's analogous to the IMPALA_CLUSTER_LOGS_DIR option. -- To view, visit http://gerrit.cloudera.org:8080/2759 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefbb2a8616adcb0cd2fb838505117e0e9ba39083 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Michael Brown <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-HasComments: Yes
