This is an automated email from the ASF dual-hosted git repository.

nic pushed a commit to branch 2.6.x
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/2.6.x by this push:
     new 615de4d  KYLIN-3998 Make bpus-min-benefit-ratio configurable in cube 
planner phase 1
615de4d is described below

commit 615de4de4d1d8a6cb205983619614435424ce797
Author: chao long <wayn...@qq.com>
AuthorDate: Fri May 10 08:12:26 2019 +0800

    KYLIN-3998 Make bpus-min-benefit-ratio configurable in cube planner phase 1
---
 .../java/org/apache/kylin/engine/mr/common/CuboidRecommenderUtil.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/CuboidRecommenderUtil.java
 
b/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/CuboidRecommenderUtil.java
index 2bffe86..10db5f6 100644
--- 
a/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/CuboidRecommenderUtil.java
+++ 
b/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/CuboidRecommenderUtil.java
@@ -60,7 +60,7 @@ public class CuboidRecommenderUtil {
 
         String key = cube.getName();
         CuboidStats cuboidStats = new CuboidStats.Builder(key, baseCuboid, 
cubeStatsReader.getCuboidRowEstimatesHLL(),
-                
cubeStatsReader.getCuboidSizeMap()).setMandatoryCuboids(mandatoryCuboids).build();
+                
cubeStatsReader.getCuboidSizeMap()).setMandatoryCuboids(mandatoryCuboids).setBPUSMinBenefitRatio(segment.getConfig().getCubePlannerBPUSMinBenefitRatio()).build();
         return 
CuboidRecommender.getInstance().getRecommendCuboidList(cuboidStats, 
segment.getConfig(),
                 !mandatoryCuboids.isEmpty());
     }

Reply via email to