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/07/21 07:34:32 UTC
kylin git commit: kylin 1788 enable arbitrary mandatory size
Repository: kylin
Updated Branches:
refs/heads/master d7cf67554 -> bacf9dc91
kylin 1788 enable arbitrary mandatory size
Signed-off-by: shaofengshi <sh...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/bacf9dc9
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/bacf9dc9
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/bacf9dc9
Branch: refs/heads/master
Commit: bacf9dc9199c5454500619de020bc78c56512216
Parents: d7cf675
Author: Cheng Wang <ch...@kyligence.io>
Authored: Wed Jul 20 18:24:53 2016 +0800
Committer: shaofengshi <sh...@apache.org>
Committed: Thu Jul 21 15:34:22 2016 +0800
----------------------------------------------------------------------
.../kylin/cube/model/validation/rule/AggregationGroupRule.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/bacf9dc9/core-cube/src/main/java/org/apache/kylin/cube/model/validation/rule/AggregationGroupRule.java
----------------------------------------------------------------------
diff --git a/core-cube/src/main/java/org/apache/kylin/cube/model/validation/rule/AggregationGroupRule.java b/core-cube/src/main/java/org/apache/kylin/cube/model/validation/rule/AggregationGroupRule.java
index ac382e9..958ac2b 100644
--- a/core-cube/src/main/java/org/apache/kylin/cube/model/validation/rule/AggregationGroupRule.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/model/validation/rule/AggregationGroupRule.java
@@ -110,8 +110,9 @@ public class AggregationGroupRule implements IValidatorRule<CubeDesc> {
int normalDimSize = normalDims.size();
int hierarchySize = count(agg.getSelectRule().hierarchy_dims);
int jointSize = count(agg.getSelectRule().joint_dims);
+ int mandatorySize = mandatoryDims.size() > 0 ? 1 : 0 ;
- if (mandatoryDims.size() + normalDimSize + hierarchySize + jointSize > maxSize) {
+ if ( mandatorySize + normalDimSize + hierarchySize + jointSize > maxSize) {
context.addResult(ResultLevel.ERROR, "Aggregation group " + index + " has too many dimensions");
continue;
}