[kylin] branch master 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 master
in repository https://gitbox.apache.org/repos/asf/kylin.git


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

commit d16bdd3a546ce7c37602471e1fdac1a3ad4e8be9
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 75d4c91..6d9b748 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
@@ -61,7 +61,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());
     }