#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 <jiatao.tao@Kyligences-MacBook-Pro.local>
Authored: Tue Jul 18 20:23:33 2017 +0800
Committer: Jiatao Tao <jiatao.tao@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);

Reply via email to