deephaven-core
deephaven-core copied to clipboard
Improve `AggFormula` performance by re-using `GroupByChunkedOperator` rather than re-creating new operators.
Initial provision is made to re-use a shared GroupByChunkedOperator in FormulaMultiColumnChunkedOperator and FormulaChunkedOperator but more work is needed before this can be fully used.
- We need to optimize the operator order in
AggregationOptimizerto look forGroupByChunkedOperatorinstances from group aggs or that may be created for other formulas. - For
FormulaMultiColumnChunkedOperator, we need to updateGroupByChunkedOperatorto keep two sets of output columns: public columns for requestedGroupByaggs and "hidden" ones only exposed to internal operators.