Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1101#discussion_r164613714
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java
---
@@ -956,21 +925,8 @@ private void spillAPartition(int part) {
this.htables[part].outputKeys(currOutBatchIndex, this.outContainer,
outStartIdxHolder.value, outNumRecordsHolder.value, numPendingOutput);
// set the value count for outgoing batch value vectors
- /* int i = 0; */
for (VectorWrapper<?> v : outgoing) {
v.getValueVector().getMutator().setValueCount(numOutputRecords);
- /*
--- End diff --
If this logic is really critical for debugging we should have utility
methods to help with it so that others can benefit from the code. Maybe we can
brainstorm about what methods would be helpful for debugging and start building
a utility library. In the meantime if you need this code to debug you can
easily look into the history and copy and paste it from there.
---
