Major Description
SortBasedAggregationIterator uses a KVIterator interface in order to process input rows as key-value pairs, but this use of KVIterator is unnecessary, slightly complicates the code, and might harm performance. We should refactor this code to remove this use of KVIterator. I'll submit a PR to do this.
