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

[ppadma]

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

    https://github.com/apache/drill/pull/1101#discussion_r166142507
  
    --- 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 --
    
    If we are using stdSize and get the value from TypeHelper, we don't need 
all value vectors to have this new function.


---