Github user Ben-Zvi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1101#discussion_r165166234
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggBatch.java
 ---
    @@ -255,7 +254,6 @@ private HashAggregator createAggregatorInternal() 
throws SchemaChangeException,
           groupByOutFieldIds[i] = container.add(vv);
         }
     
    -    int extraNonNullColumns = 0; // each of SUM, MAX and MIN gets an extra 
bigint column
    --- End diff --
    
    Maybe do this work as a separate PR (for DRILL-5728) ?  Else it would delay 
this PR, and overload it ...


---

Reply via email to