This is an automated email from the ASF dual-hosted git repository. nju_yaho 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 601deae KYLIN-3580 Avoid using entrySet to add elements 601deae is described below commit 601deae4e72203c3309023848d70cd95941029c2 Author: kyotoYaho <nju_y...@apache.org> AuthorDate: Mon Jan 7 20:38:19 2019 +0800 KYLIN-3580 Avoid using entrySet to add elements --- .../java/org/apache/kylin/cube/cuboid/algorithm/CuboidStatsUtil.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/core-cube/src/main/java/org/apache/kylin/cube/cuboid/algorithm/CuboidStatsUtil.java b/core-cube/src/main/java/org/apache/kylin/cube/cuboid/algorithm/CuboidStatsUtil.java index ec2f496..d9aaf54 100644 --- a/core-cube/src/main/java/org/apache/kylin/cube/cuboid/algorithm/CuboidStatsUtil.java +++ b/core-cube/src/main/java/org/apache/kylin/cube/cuboid/algorithm/CuboidStatsUtil.java @@ -150,7 +150,10 @@ public class CuboidStatsUtil { return ret == 0 ? o1.getKey().compareTo(o2.getKey()) : ret; } }); - sortedStatsSet.addAll(statistics.entrySet()); + //sortedStatsSet.addAll(statistics.entrySet()); KYLIN-3580 + for (Map.Entry<Long, Long> entry : statistics.entrySet()) { + sortedStatsSet.add(entry); + } for (Long cuboid : cuboids) { if (statistics.get(cuboid) == null) { // Get estimate row count for mandatory cuboid