[kylin] branch master updated: KYLIN-3399, makes the code that added in KYLIN-2474 more intuitive.

[billyliu]

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

billyliu 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 d680a4a  KYLIN-3399, makes the code that added in KYLIN-2474 more 
intuitive.
d680a4a is described below

commit d680a4aa9162fdcf0e845bde7b6952aeeac81410
Author: tttMelody <245915...@qq.com>
AuthorDate: Wed Jun 20 11:25:03 2018 +0800

    KYLIN-3399, makes the code that added in KYLIN-2474 more intuitive.
---
 .../java/org/apache/kylin/cube/cli/DictionaryGeneratorCLI.java    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git 
a/core-cube/src/main/java/org/apache/kylin/cube/cli/DictionaryGeneratorCLI.java 
b/core-cube/src/main/java/org/apache/kylin/cube/cli/DictionaryGeneratorCLI.java
index 2a24370..7fcf320 100644
--- 
a/core-cube/src/main/java/org/apache/kylin/cube/cli/DictionaryGeneratorCLI.java
+++ 
b/core-cube/src/main/java/org/apache/kylin/cube/cli/DictionaryGeneratorCLI.java
@@ -94,8 +94,12 @@ public class DictionaryGeneratorCLI {
         cubeSeg = updatedCube.getSegmentById(cubeSeg.getUuid());
         for (TableRef lookup : toCheckLookup) {
             logger.info("Checking snapshot of " + lookup);
-            JoinDesc join = 
cubeSeg.getModel().getJoinsTree().getJoinByPKSide(lookup);
-            cubeMgr.getLookupTable(cubeSeg, join);
+            try {
+                JoinDesc join = 
cubeSeg.getModel().getJoinsTree().getJoinByPKSide(lookup);
+                cubeMgr.getLookupTable(cubeSeg, join);
+            } catch (Throwable th) {
+                throw new RuntimeException("Checking snapshot of " + lookup + 
" failed.", th);
+            }
         }
     }