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 <dam6...@gmail.com> Authored: Mon Mar 19 08:53:45 2018 -0700 Committer: Sahil Takiar <stak...@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