[GitHub] drill pull request #1101: DRILL-6032: Made the batch sizing for HashAgg more...

[ilooner]

Github user ilooner commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1101#discussion_r165137630
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/spill/RecordBatchSizer.java
 ---
    @@ -232,9 +251,8 @@ else if (width > 0) {
         }
       }
     
    -  public static final int MAX_VECTOR_SIZE = ValueVector.MAX_BUFFER_SIZE; 
// 16 MiB
    -
       private List<ColumnSize> columnSizes = new ArrayList<>();
    +  private Map<String, ColumnSize> columnSizeMap = 
CaseInsensitiveMap.newHashMap();
    --- End diff --
    
    Thanks for the explanation here and on the dev list @paul-rogers. 


---