You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by sh...@apache.org on 2016/09/19 15:17:53 UTC
kylin git commit: Revert "KYLIN-2032 do not need dict for partition
col if it is not in dimension"
Repository: kylin
Updated Branches:
refs/heads/master a8a1b6e03 -> e17b11bd8
Revert "KYLIN-2032 do not need dict for partition col if it is not in dimension"
This reverts commit a8a1b6e037767b75a3184f4b5db6f384f622a89e.
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/e17b11bd
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/e17b11bd
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/e17b11bd
Branch: refs/heads/master
Commit: e17b11bd861a1ce7ecfda37997e0011f7e180fd2
Parents: a8a1b6e
Author: shaofengshi <sh...@apache.org>
Authored: Mon Sep 19 22:58:12 2016 +0800
Committer: shaofengshi <sh...@apache.org>
Committed: Mon Sep 19 22:58:12 2016 +0800
----------------------------------------------------------------------
.../src/main/java/org/apache/kylin/cube/CubeManager.java | 7 +++++++
1 file changed, 7 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/e17b11bd/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
----------------------------------------------------------------------
diff --git a/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java b/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
index 2660c9f..57b9510 100644
--- a/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
@@ -943,6 +943,13 @@ public class CubeManager implements IRealizationProvider {
}
}
+ // add partition column in all case
+ if (cubeDesc.getModel().getPartitionDesc() != null) {
+ TblColRef partitionCol = cubeDesc.getModel().getPartitionDesc().getPartitionDateColumnRef();
+ if (factDictCols.contains(partitionCol) == false) {
+ factDictCols.add(partitionCol);
+ }
+ }
return factDictCols;
}
}