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 2015/09/03 03:31:50 UTC
[35/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/560ab43a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kylin/tree/560ab43a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kylin/diff/560ab43a
Branch: refs/heads/0.7
Commit: 560ab43a47d9781f9fc7eab668cef64bbb4e5292
Parents: 294a3eb
Author: shaofengshi <sh...@apache.org>
Authored: Thu Aug 27 14:23:14 2015 +0800
Committer: shaofengshi <sh...@apache.org>
Committed: Fri Aug 28 11:02:10 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/560ab43a/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;
}