Repository: kylin
Updated Branches:
  refs/heads/yang21-cdh5.7 1da87a6ef -> d205a506d (forced update)


KYLIN-2097 Get 'Column does not exist in row key desc" on cube has TopN measure

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

Branch: refs/heads/yang21-cdh5.7
Commit: 34b6419d77bd7d01dfb1e44615c211286ae60a0f
Parents: 6ff422d
Author: shaofengshi <shaofeng...@apache.org>
Authored: Mon Oct 17 12:22:45 2016 +0800
Committer: shaofengshi <shaofeng...@apache.org>
Committed: Thu Oct 20 14:36:29 2016 +0800

----------------------------------------------------------------------
 .../java/org/apache/kylin/measure/topn/TopNMeasureType.java     | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/34b6419d/core-metadata/src/main/java/org/apache/kylin/measure/topn/TopNMeasureType.java
----------------------------------------------------------------------
diff --git 
a/core-metadata/src/main/java/org/apache/kylin/measure/topn/TopNMeasureType.java
 
b/core-metadata/src/main/java/org/apache/kylin/measure/topn/TopNMeasureType.java
index 800ca88..b0d469d 100644
--- 
a/core-metadata/src/main/java/org/apache/kylin/measure/topn/TopNMeasureType.java
+++ 
b/core-metadata/src/main/java/org/apache/kylin/measure/topn/TopNMeasureType.java
@@ -246,6 +246,11 @@ public class TopNMeasureType extends 
MeasureType<TopNCounter<ByteArray>> {
         if (digest.groupbyColumns.containsAll(literalCol) == false)
             return null;
 
+        for (TblColRef colRef : literalCol) {
+            if (digest.filterColumns.contains(colRef) == true) {
+                return null;
+            }
+        }
         unmatchedDimensions.removeAll(literalCol);
         unmatchedAggregations.remove(onlyFunction);
         return new CapabilityInfluence() {

Reply via email to