[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_r166388922
  
    --- Diff: exec/vector/src/main/codegen/templates/FixedValueVectors.java ---
    @@ -298,6 +298,11 @@ public int getPayloadByteCount(int valueCount) {
         return valueCount * ${type.width};
       }
     
    +  @Override
    +  public int getValueWidth() {
    --- End diff --
    
    TypeHelper doesn't return the correct size for Nullable columns. For 
example NullableIntVector has a width of 5 instead of 4.


---