Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/938#discussion_r139045903
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggregator.java
---
@@ -47,10 +47,7 @@
// OK - batch returned, NONE - end of data, RESTART - call again
public enum AggIterOutcome { AGG_OK, AGG_NONE, AGG_RESTART }
- public abstract void setup(HashAggregate hashAggrConfig, HashTableConfig
htConfig, FragmentContext context,
- OperatorStats stats, OperatorContext
oContext, RecordBatch incoming, HashAggBatch outgoing,
- LogicalExpression[] valueExprs,
List<TypedFieldId> valueFieldIds, TypedFieldId[] keyFieldIds,
- VectorContainer outContainer) throws
SchemaChangeException, IOException, ClassTransformationException;
+ public abstract void setup(HashAggregate hashAggrConfig, HashTableConfig
htConfig, FragmentContext context, OperatorStats stats, OperatorContext
oContext, RecordBatch incoming, HashAggBatch outgoing, LogicalExpression[]
valueExprs, List<TypedFieldId> valueFieldIds, TypedFieldId[] keyFieldIds,
VectorContainer outContainer, int extraRowBytes) throws SchemaChangeException,
IOException, ClassTransformationException;
--- End diff --
That was one of the IDE's ideas ....
And simplification could be done as part of future cleanup work (like
DRILL-5779)
---
