[druid] branch master updated: low hanging fruit - presize hash map for DruidSegmentReader (#9836)

[jonwei]

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

jonwei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new 267a6cc  low hanging fruit - presize hash map for DruidSegmentReader 
(#9836)
267a6cc is described below

commit 267a6cc175736dccb16e8e43abd2772c0f44f611
Author: Clint Wylie <cwy...@apache.org>
AuthorDate: Thu May 7 12:39:14 2020 -0700

    low hanging fruit - presize hash map for DruidSegmentReader (#9836)
---
 .../java/org/apache/druid/indexing/input/DruidSegmentReader.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/indexing-service/src/main/java/org/apache/druid/indexing/input/DruidSegmentReader.java
 
b/indexing-service/src/main/java/org/apache/druid/indexing/input/DruidSegmentReader.java
index f2ce056..6460ae4 100644
--- 
a/indexing-service/src/main/java/org/apache/druid/indexing/input/DruidSegmentReader.java
+++ 
b/indexing-service/src/main/java/org/apache/druid/indexing/input/DruidSegmentReader.java
@@ -21,7 +21,6 @@ package org.apache.druid.indexing.input;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
-import com.google.common.collect.Maps;
 import org.apache.druid.data.input.InputEntity;
 import org.apache.druid.data.input.InputEntity.CleanableFile;
 import org.apache.druid.data.input.InputRow;
@@ -49,6 +48,7 @@ import org.apache.druid.segment.column.ColumnHolder;
 import org.apache.druid.segment.data.IndexedInts;
 import org.apache.druid.segment.filter.Filters;
 import org.apache.druid.segment.realtime.firehose.WindowedStorageAdapter;
+import org.apache.druid.utils.CollectionUtils;
 import org.joda.time.DateTime;
 
 import java.io.File;
@@ -245,7 +245,8 @@ public class DruidSegmentReader extends 
IntermediateRowParsingReader<Map<String,
       if (!hasNext()) {
         throw new NoSuchElementException();
       }
-      final Map<String, Object> theEvent = Maps.newLinkedHashMap();
+      final Map<String, Object> theEvent =
+          CollectionUtils.newLinkedHashMapWithExpectedSize(dimSelectors.size() 
+ metSelectors.size() + 1);
 
       for (Entry<String, DimensionSelector> dimSelector : 
dimSelectors.entrySet()) {
         final String dim = dimSelector.getKey();


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org