KYLIN-2603 enable GTAggregateScanner when there is having filter

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

Branch: refs/heads/KYLIN-2606
Commit: 0e44a195b5b2a0074896ffae0636b82a1d7c1355
Parents: e1bc72e
Author: Li Yang <liy...@apache.org>
Authored: Thu May 11 17:26:10 2017 +0800
Committer: hongbin ma <m...@kyligence.io>
Committed: Tue May 23 20:18:17 2017 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/gridtable/GTScanRequest.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/0e44a195/core-cube/src/main/java/org/apache/kylin/gridtable/GTScanRequest.java
----------------------------------------------------------------------
diff --git 
a/core-cube/src/main/java/org/apache/kylin/gridtable/GTScanRequest.java 
b/core-cube/src/main/java/org/apache/kylin/gridtable/GTScanRequest.java
index 9b6b2a6..ffaa8bd 100644
--- a/core-cube/src/main/java/org/apache/kylin/gridtable/GTScanRequest.java
+++ b/core-cube/src/main/java/org/apache/kylin/gridtable/GTScanRequest.java
@@ -195,7 +195,7 @@ public class GTScanRequest {
                 return new EmptyGTScanner();
             }
 
-            if (!this.isAllowStorageAggregation()) {
+            if (!this.isAllowStorageAggregation() && havingFilterPushDown == 
null) {
                 logger.info("pre aggregation is not beneficial, skip it");
             } else if (this.hasAggregation()) {
                 logger.info("pre aggregating results before returning");

Reply via email to