yashmayya opened a new pull request, #17815:
URL: https://github.com/apache/pinot/pull/17815
- `EmptyResponseUtils` was iterating over individual aggregation functions
instead of select expressions, so queries with
post-aggregation expressions (e.g., `SUM(a) * 1.0 / COUNT(*)`) produced
the wrong number of columns with wrong names
- The normal (non-empty) code path already handled this correctly via
`PostAggregationHandler`
- This patch rewrites `buildEmptyAggregationResultTable` and
`buildEmptyGroupByResultTable` to use `PostAggregationHandler`, matching the
pattern in `AggregationDataTableReducer` and `GroupByDataTableReducer`
- Extracted alias application into a shared `applyAliases` helper that
runs after `PostAggregationHandler` produces the correct schema.
- Added tests for post-aggregation expressions with and without aliases,
including a GROUP BY variant.
- This fixes an edge case missed in
https://github.com/apache/pinot/pull/17143/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
