You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by ma...@apache.org on 2016/07/27 01:56:54 UTC
kylin git commit: KYLIN-1913 use corrected sql for final query logging
Repository: kylin
Updated Branches:
refs/heads/beforepomupdate 5e275840f -> a055b8891
KYLIN-1913 use corrected sql for final query logging
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/a055b889
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/a055b889
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/a055b889
Branch: refs/heads/beforepomupdate
Commit: a055b8891c7f6c5ae76b8aca7e2cb4aee5d75edc
Parents: 5e27584
Author: Hongbin Ma <ma...@apache.org>
Authored: Wed Jul 27 09:56:38 2016 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Wed Jul 27 09:56:38 2016 +0800
----------------------------------------------------------------------
.../main/java/org/apache/kylin/rest/service/QueryService.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/a055b889/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 84a5c67..cb4df0d 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
@@ -267,8 +267,10 @@ public class QueryService extends BasicService {
}
String correctedSql = QueryUtil.massageSql(sqlRequest);
- if (correctedSql.equals(sqlRequest.getSql()) == false)
+ if (!correctedSql.equals(sqlRequest.getSql())) {
logger.info("The corrected query: " + correctedSql);
+ sqlRequest.setSql(correctedSql);
+ }
// add extra parameters into olap context, like acceptPartial
Map<String, String> parameters = new HashMap<String, String>();