Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1125#discussion_r172105268
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatchSizer.java 
---
    @@ -525,4 +763,11 @@ public VectorInitializer buildVectorInitializer() {
         }
         return initializer;
       }
    +
    +  public void allocateVectors(VectorContainer container, int recordCount) {
    +    for (VectorWrapper w : container) {
    +      ColumnSize colSize = columnSizes.get(w.getField().getName());
    +      colSize.allocateVector(w.getValueVector(), recordCount);
    +    }
    +  }
    --- End diff --
    
    Very nice clean-up and enhancements. Now I don't feel so bad about this 
stuff being quick & dirty...


---

Reply via email to