You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by lu...@apache.org on 2015/09/06 09:59:04 UTC
[32/50] [abbrv] incubator-kylin git commit: Small update in seeking
the log file path
Small update in seeking the log file path
Project: http://git-wip-us.apache.org/repos/asf/incubator-kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kylin/commit/1e8cd386
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kylin/tree/1e8cd386
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kylin/diff/1e8cd386
Branch: refs/heads/0.7-staging
Commit: 1e8cd386e681ac9a031ed8950355983828f18915
Parents: ad0a1d1
Author: shaofengshi <sh...@apache.org>
Authored: Thu Aug 27 14:23:14 2015 +0800
Committer: Luke Han <lu...@apache.org>
Committed: Sun Sep 6 14:37:58 2015 +0800
----------------------------------------------------------------------
.../java/org/apache/kylin/monitor/ConfigUtils.java | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kylin/blob/1e8cd386/monitor/src/main/java/org/apache/kylin/monitor/ConfigUtils.java
----------------------------------------------------------------------
diff --git a/monitor/src/main/java/org/apache/kylin/monitor/ConfigUtils.java b/monitor/src/main/java/org/apache/kylin/monitor/ConfigUtils.java
index 916e1c8..93cb00f 100644
--- a/monitor/src/main/java/org/apache/kylin/monitor/ConfigUtils.java
+++ b/monitor/src/main/java/org/apache/kylin/monitor/ConfigUtils.java
@@ -154,14 +154,14 @@ public class ConfigUtils {
}
String kylinHome = getKylinHome();
- if (!StringUtils.isEmpty(kylinHome))
- if (logDirList.isEmpty()) {
- throw new RuntimeException("Didn't find KYLIN_CONF or KYLIN_HOME or KYLIN_EXT_LOG_BASE_DIR, please set one of them");
- } else {
- String path = kylinHome + File.separator + "tomcat" + File.separator + "logs";
- logDirList.add(path);
- }
-
+ if (!StringUtils.isEmpty(kylinHome)) {
+ String path = kylinHome + File.separator + "tomcat" + File.separator + "logs";
+ logDirList.add(path);
+ }
+
+ if (logDirList.isEmpty())
+ throw new RuntimeException("Didn't find KYLIN_CONF or KYLIN_HOME or KYLIN_EXT_LOG_BASE_DIR, please set one of them");
+
return logDirList;
}