You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by st...@apache.org on 2018/03/19 15:54:08 UTC
hive git commit: HIVE-18344: Remove LinkedList from
SharedWorkOptimizer.java (BELUGA BEHR, reviewed by Sahil Takiar)
Repository: hive
Updated Branches:
refs/heads/master b7c1aa63b -> 79e88695c
HIVE-18344: Remove LinkedList from SharedWorkOptimizer.java (BELUGA BEHR, reviewed by Sahil Takiar)
Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/79e88695
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/79e88695
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/79e88695
Branch: refs/heads/master
Commit: 79e88695cfa47749435e7e793c73697cc3f4a90e
Parents: b7c1aa6
Author: BELUGA BEHR <da...@gmail.com>
Authored: Mon Mar 19 08:53:45 2018 -0700
Committer: Sahil Takiar <st...@cloudera.com>
Committed: Mon Mar 19 08:53:45 2018 -0700
----------------------------------------------------------------------
.../apache/hadoop/hive/ql/optimizer/SharedWorkOptimizer.java | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hive/blob/79e88695/ql/src/java/org/apache/hadoop/hive/ql/optimizer/SharedWorkOptimizer.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/optimizer/SharedWorkOptimizer.java b/ql/src/java/org/apache/hadoop/hive/ql/optimizer/SharedWorkOptimizer.java
index bc6d0bb..b60512b 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/optimizer/SharedWorkOptimizer.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/optimizer/SharedWorkOptimizer.java
@@ -27,7 +27,6 @@ import java.util.HashMap;
import java.util.HashSet;
import java.util.Iterator;
import java.util.LinkedHashSet;
-import java.util.LinkedList;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;
@@ -595,7 +594,7 @@ public class SharedWorkOptimizer extends Transform {
}
}
List<Entry<String, Long>> sortedTables =
- new LinkedList<>(tableToTotalSize.entrySet());
+ new ArrayList<>(tableToTotalSize.entrySet());
Collections.sort(sortedTables, Collections.reverseOrder(
new Comparator<Map.Entry<String, Long>>() {
@Override
@@ -635,7 +634,7 @@ public class SharedWorkOptimizer extends Transform {
StatsUtils.safeMult(op.getChildOperators().size(), size));
}
List<Entry<Operator<?>, Long>> sortedOps =
- new LinkedList<>(opToTotalSize.entrySet());
+ new ArrayList<>(opToTotalSize.entrySet());
Collections.sort(sortedOps, Collections.reverseOrder(
new Comparator<Map.Entry<Operator<?>, Long>>() {
@Override