Repository: kylin Updated Branches: refs/heads/2.2.x fb2580808 -> dd0630775
Throw exception from pushdown instead of exception from calcite Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/da0d1535 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/da0d1535 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/da0d1535 Branch: refs/heads/2.2.x Commit: da0d1535c5c6cd682f3da98cd6961a271d5aaca5 Parents: 53cad89 Author: nichunen <chunen...@kyligence.io> Authored: Wed Oct 18 20:31:54 2017 +0800 Committer: nichunen <chunen...@kyligence.io> Committed: Wed Oct 18 20:31:54 2017 +0800 ---------------------------------------------------------------------- .../main/java/org/apache/kylin/rest/service/QueryService.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/da0d1535/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java ---------------------------------------------------------------------- diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java b/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java index ddb805c..c6a9b76 100644 --- a/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java +++ b/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java @@ -864,8 +864,9 @@ public class QueryService extends BasicService { r = PushDownUtil.tryPushDownSelectQuery(sqlRequest.getProject(), correctedSql, conn.getSchema(), sqlException); } catch (Exception e2) { - //exception in pushdown engine will be printed, but we'll not re-throw it, we'll - logger.info("pushdown engine failed current query too", e2); + logger.error("pushdown engine failed current query too", e2); + //exception in pushdown, throw it instead of exception in calcite + throw e2; } if (r == null)
