KYLIN-2574 RawQueryLastHacker should group by all possible dimensions

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

Branch: refs/heads/KYLIN-2624
Commit: d6fdda2c24ac89ec72ff9663a36993f42a70960b
Parents: 1ba9af7
Author: Hongbin Ma <mahong...@apache.org>
Authored: Fri Apr 28 15:03:43 2017 +0800
Committer: liyang-gmt8 <liy...@apache.org>
Committed: Mon May 8 16:01:48 2017 +0800

----------------------------------------------------------------------
 .../main/java/org/apache/kylin/cube/RawQueryLastHacker.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/d6fdda2c/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
----------------------------------------------------------------------
diff --git 
a/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java 
b/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
index 682e48c..d2e3a83 100644
--- a/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
@@ -46,8 +46,8 @@ public class RawQueryLastHacker {
         boolean isSelectAll = sqlDigest.allColumns.isEmpty() || 
sqlDigest.allColumns.equals(sqlDigest.filterColumns);
         for (TblColRef col : cubeDesc.listAllColumns()) {
             if (cubeDesc.listDimensionColumnsIncludingDerived().contains(col) 
|| isSelectAll) {
-                if (tupleInfo.hasColumn(col))
-                    sqlDigest.allColumns.add(col);
+                //if (tupleInfo.hasColumn(col))
+                sqlDigest.allColumns.add(col);
             }
         }
 
@@ -77,4 +77,4 @@ public class RawQueryLastHacker {
             }
         }
     }
-}
\ No newline at end of file
+}

Reply via email to