You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/04/06 11:17:07 UTC
kylin git commit: minor, set default compress in diagnosis tool
Repository: kylin
Updated Branches:
refs/heads/master b37c5bb29 -> accbb5420
minor, set default compress in diagnosis tool
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/accbb542
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/accbb542
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/accbb542
Branch: refs/heads/master
Commit: accbb5420bc0fc3a9768a69780f2d02977ab775e
Parents: b37c5bb
Author: lidongsjtu <li...@apache.org>
Authored: Wed Apr 6 17:16:47 2016 +0800
Committer: lidongsjtu <li...@apache.org>
Committed: Wed Apr 6 17:16:52 2016 +0800
----------------------------------------------------------------------
tool/src/main/java/org/apache/kylin/tool/DiagnosisInfoCLI.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/accbb542/tool/src/main/java/org/apache/kylin/tool/DiagnosisInfoCLI.java
----------------------------------------------------------------------
diff --git a/tool/src/main/java/org/apache/kylin/tool/DiagnosisInfoCLI.java b/tool/src/main/java/org/apache/kylin/tool/DiagnosisInfoCLI.java
index e214a6f..e0e3f2f 100644
--- a/tool/src/main/java/org/apache/kylin/tool/DiagnosisInfoCLI.java
+++ b/tool/src/main/java/org/apache/kylin/tool/DiagnosisInfoCLI.java
@@ -47,7 +47,7 @@ public class DiagnosisInfoCLI extends AbstractApplication {
private static final Option OPTION_LOG_PERIOD = OptionBuilder.withArgName("logPeriod").hasArg().isRequired(false).withDescription("specify how many days of kylin logs to extract. Default 3.").create("logPeriod");
@SuppressWarnings("static-access")
- private static final Option OPTION_COMPRESS = OptionBuilder.withArgName("compress").hasArg().isRequired(false).withDescription("specify whether to compress the output with zip. Default false.").create("compress");
+ private static final Option OPTION_COMPRESS = OptionBuilder.withArgName("compress").hasArg().isRequired(false).withDescription("specify whether to compress the output with zip. Default true.").create("compress");
@SuppressWarnings("static-access")
private static final Option OPTION_DEST = OptionBuilder.withArgName("destDir").hasArg().isRequired(true).withDescription("specify the dest dir to save the related metadata").create("destDir");
@@ -103,7 +103,7 @@ public class DiagnosisInfoCLI extends AbstractApplication {
cubeMetaExtractor.execute(cubeMetaArgs);
int logPeriod = optionsHelper.hasOption(OPTION_LOG_PERIOD) ? Integer.valueOf(optionsHelper.getOptionValue(OPTION_LOG_PERIOD)) : DEFAULT_LOG_PERIOD;
- boolean compress = optionsHelper.hasOption(OPTION_COMPRESS) ? Boolean.valueOf(optionsHelper.getOptionValue(OPTION_COMPRESS)) : false;
+ boolean compress = optionsHelper.hasOption(OPTION_COMPRESS) ? Boolean.valueOf(optionsHelper.getOptionValue(OPTION_COMPRESS)) : true;
boolean includeConf = optionsHelper.hasOption(OPTION_INCLUDE_CONF) ? Boolean.valueOf(optionsHelper.getOptionValue(OPTION_INCLUDE_CONF)) : true;
// export logs