From 1d0a184471fe4edd7b7a8775c3b65173e4db564c Mon Sep 17 00:00:00 2001 From: Cheng Wang Date: Wed, 20 Jul 2016 18:24:53 +0800 Subject: [PATCH] kylin 1788 enable arbitrary mandatory size --- .../apache/kylin/cube/model/validation/rule/AggregationGroupRule.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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..0475b77 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 { 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; } -- 2.7.4 (Apple Git-66)