[kylin] branch master updated: KYLIN-3981 Auto Merge Job failed to execute on windows

[ni...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

nic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/master by this push:
     new c04694b  KYLIN-3981 Auto Merge Job failed to execute on windows
c04694b is described below

commit c04694bf9a7e70e7f63870157884fdcde030f88a
Author: GinaZhai <na...@kyligence.io>
AuthorDate: Thu Apr 25 16:32:02 2019 +0800

    KYLIN-3981 Auto Merge Job failed to execute on windows
---
 core-common/src/main/java/org/apache/kylin/common/KylinConfig.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/core-common/src/main/java/org/apache/kylin/common/KylinConfig.java b/core-common/src/main/java/org/apache/kylin/common/KylinConfig.java
index 9aa54d6..f7c6033 100644
--- a/core-common/src/main/java/org/apache/kylin/common/KylinConfig.java
+++ b/core-common/src/main/java/org/apache/kylin/common/KylinConfig.java
@@ -42,6 +42,7 @@ import org.apache.kylin.common.restclient.RestClient;
 import org.apache.kylin.common.threadlocal.InternalThreadLocal;
 import org.apache.kylin.common.util.ClassUtil;
 import org.apache.kylin.common.util.OrderedProperties;
+import org.apache.zookeeper.Shell;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -184,7 +185,8 @@ public class KylinConfig extends KylinConfigBase {
 
         try {
             File file = new File(metaUri);
-            if (file.exists() || metaUri.contains("/")) {
+            // for the developers using windows, without this condition, it will never find the file
+            if (file.exists() || metaUri.contains("/") || Shell.WINDOWS) {
                 if (!file.exists()) {
                     file.mkdirs();
                 }