minor, fix agg combination bug
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/5eca7f6b Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/5eca7f6b Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/5eca7f6b Branch: refs/heads/KYLIN-2606 Commit: 5eca7f6b59454df4681deb17f361d4708a2affca Parents: 40054dd Author: Roger Shi <rogershijich...@hotmail.com> Authored: Thu Jun 29 23:07:16 2017 +0800 Committer: liyang-gmt8 <liy...@apache.org> Committed: Thu Jun 29 23:11:00 2017 +0800 ---------------------------------------------------------------------- .../main/java/org/apache/kylin/cube/model/AggregationGroup.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/5eca7f6b/core-cube/src/main/java/org/apache/kylin/cube/model/AggregationGroup.java ---------------------------------------------------------------------- diff --git a/core-cube/src/main/java/org/apache/kylin/cube/model/AggregationGroup.java b/core-cube/src/main/java/org/apache/kylin/cube/model/AggregationGroup.java index 064d657..0533ea1 100644 --- a/core-cube/src/main/java/org/apache/kylin/cube/model/AggregationGroup.java +++ b/core-cube/src/main/java/org/apache/kylin/cube/model/AggregationGroup.java @@ -320,7 +320,7 @@ public class AggregationGroup implements Serializable { normalDims.removeAll(jointDims); combination = combination * (1L << normalDims.size()); - if (cubeDesc.getConfig().getCubeAggrGroupIsMandatoryOnlyValid()) { + if (cubeDesc.getConfig().getCubeAggrGroupIsMandatoryOnlyValid() && !mandatoryDims.isEmpty()) { combination += 1; } combination -= 1; // not include cuboid 0
