[kylin] branch 2.6.x updated: KYLIN-3998 Make bpus-min-benefit-ratio configurable in cube planner phase 1

[ni...@apache.org]

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 <wa...@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());
     }