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 2018/03/11 06:28:12 UTC

[kylin] 03/04: KYLIN-3271 Minor, avoid null pointer exception

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

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

commit c2ef3c62a96ca52ef673335f621232d6d664c24a
Author: nichunen <ch...@kyligence.io>
AuthorDate: Wed Mar 7 11:50:11 2018 +0800

    KYLIN-3271 Minor, avoid null pointer exception
---
 .../main/java/org/apache/kylin/common/persistence/ResourceTool.java  | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/core-common/src/main/java/org/apache/kylin/common/persistence/ResourceTool.java b/core-common/src/main/java/org/apache/kylin/common/persistence/ResourceTool.java
index f270743..8803320 100644
--- a/core-common/src/main/java/org/apache/kylin/common/persistence/ResourceTool.java
+++ b/core-common/src/main/java/org/apache/kylin/common/persistence/ResourceTool.java
@@ -174,7 +174,10 @@ public class ResourceTool {
         logger.info("Copy from {} to {}", src, dst);
 
         for (String resourceRoot : SKIP_CHILDREN_CHECK_RESOURCE_ROOT) {
-            pathsSkipChildrenCheck.addAll(src.listResourcesRecursively(resourceRoot));
+            NavigableSet<String> all = src.listResourcesRecursively(resourceRoot);
+            if (all != null) {
+                pathsSkipChildrenCheck.addAll(src.listResourcesRecursively(resourceRoot));
+            }
         }
         copyR(src, dst, path, copyImmutableResource);
     }

-- 
To stop receiving notification emails like this one, please contact
liyang@apache.org.