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

[shaofengshi]

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

shaofengshi 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 043a315  KYLIN-3580 Avoid using entrySet to add 
elements@CuboidStatsUtil#complementRowCountForMandatoryCuboids
043a315 is described below

commit 043a3156c17f9c2dbd76125ab8e957d29c1dffed
Author: Lijun Cao <641507...@qq.com>
AuthorDate: Tue Sep 25 17:03:19 2018 +0800

    KYLIN-3580 Avoid using entrySet to add 
elements@CuboidStatsUtil#complementRowCountForMandatoryCuboids
---
 .../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 c2683df..def1e68 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
@@ -87,7 +87,10 @@ public class CuboidStatsUtil {
                         return o1.getValue().compareTo(o2.getValue());
                     }
                 });
-        sortedStatsSet.addAll(statistics.entrySet());
+        //sortedStatsSet.addAll(statistics.entrySet()); KYLIN-3580
+        for(Map.Entry<Long, Long> entry : statistics.entrySet()){
+            sortedStatsSet.add(entry);
+        }
         for (Long cuboid : mandatoryCuboidSet) {
             if (statistics.get(cuboid) == null) {
                 // Get estimate row count for mandatory cuboid