[kylin] branch master updated: KYLIN-3580 Avoid using entrySet to add elements

[nj...@apache.org]

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 <nj...@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