You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by bi...@apache.org on 2018/06/20 08:54:48 UTC
[kylin] branch master updated: KYLIN-3399,
makes the code that added in KYLIN-2474 more intuitive.
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 <24...@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);
+ }
}
}