Skip to content

Commit

Permalink
DRILL-5121 Fix for memory leak. Changes fieldVectorMap in ScanBatch t…
Browse files Browse the repository at this point in the history
…o a CaseInsensitiveMap

close #690
  • Loading branch information
karthik authored and Aman Sinha committed Jan 8, 2017
1 parent 167f0ce commit 15b021f
Showing 1 changed file with 3 additions and 2 deletions.
Expand Up @@ -53,6 +53,7 @@
import org.apache.drill.exec.vector.NullableVarCharVector; import org.apache.drill.exec.vector.NullableVarCharVector;
import org.apache.drill.exec.vector.SchemaChangeCallBack; import org.apache.drill.exec.vector.SchemaChangeCallBack;
import org.apache.drill.exec.vector.ValueVector; import org.apache.drill.exec.vector.ValueVector;
import org.apache.drill.common.map.CaseInsensitiveMap;


import com.google.common.collect.Maps; import com.google.common.collect.Maps;


Expand All @@ -67,8 +68,8 @@ public class ScanBatch implements CloseableRecordBatch {
private final VectorContainer container = new VectorContainer(); private final VectorContainer container = new VectorContainer();


/** Fields' value vectors indexed by fields' keys. */ /** Fields' value vectors indexed by fields' keys. */
private final Map<String, ValueVector> fieldVectorMap = private final CaseInsensitiveMap<ValueVector> fieldVectorMap =
Maps.newHashMap(); CaseInsensitiveMap.newHashMap();


private int recordCount; private int recordCount;
private final FragmentContext context; private final FragmentContext context;
Expand Down

0 comments on commit 15b021f

Please sign in to comment.