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 2017/07/25 09:44:56 UTC

[09/24] kylin git commit: #1355 bugfix fix swallow exception and exit the method

#1355 bugfix fix swallow exception and exit the method


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/6252ae34
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/6252ae34
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/6252ae34

Branch: refs/heads/2.1.x
Commit: 6252ae34564232619181b38fd5fedc952da4382f
Parents: 24fe421
Author: Jiatao Tao <ji...@Kyligences-MacBook-Pro.local>
Authored: Tue Jul 18 20:23:33 2017 +0800
Committer: Jiatao Tao <ji...@Kyligences-MacBook-Pro.local>
Committed: Tue Jul 18 20:39:26 2017 +0800

----------------------------------------------------------------------
 .../java/org/apache/kylin/query/util/ConvertToComputedColumn.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/6252ae34/query/src/main/java/org/apache/kylin/query/util/ConvertToComputedColumn.java
----------------------------------------------------------------------
diff --git a/query/src/main/java/org/apache/kylin/query/util/ConvertToComputedColumn.java b/query/src/main/java/org/apache/kylin/query/util/ConvertToComputedColumn.java
index a9ff360..c9feaab 100644
--- a/query/src/main/java/org/apache/kylin/query/util/ConvertToComputedColumn.java
+++ b/query/src/main/java/org/apache/kylin/query/util/ConvertToComputedColumn.java
@@ -82,7 +82,8 @@ public class ConvertToComputedColumn implements QueryUtil.IQueryTransformer {
             try {
                 matchedNodes = getMatchedNodes(inputSql, computedColumn.get(ccExp));
             } catch (SqlParseException e) {
-                logger.error("Convert to computedColumn Fail,parse sql fail ", e.getMessage());
+                logger.error("Convert to computedColumn Fail,parse sql fail ", e);
+                return inputSql;
             }
             for (SqlNode node : matchedNodes) {
                 Pair<Integer, Integer> startEndPos = CalciteParser.getReplacePos(node, lines);