Github user ppadma commented on a diff in the pull request:
https://github.com/apache/drill/pull/1101#discussion_r164599681
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java
---
@@ -215,6 +206,7 @@ public BatchHolder() {
MaterializedField outputField = materializedValueFields[i];
// Create a type-specific ValueVector for this value
vector = TypeHelper.getNewVector(outputField, allocator);
+ int columnSize = new RecordBatchSizer.ColumnSize(vector).estSize;
--- End diff --
I wonder what is wrong if estSize is 0 when there is no data. If there is
no data for a column, why would we want to add it's value width to outgoing row
width ?
---
