cryptoe opened a new pull request, #17869: URL: https://github.com/apache/druid/pull/17869
* fixes and validation for projections with rollup base tables changes: * Moved projection dimension initialization into `OnHeapAggregateProjection` instead of being done externally * Fix projection aggregations to work properly for rollup base tables with the same aggregations (swap out projection agg with base table agg to build the projection from the raw input row) * Added validation for projection aggregations to ensure they either reference a base table aggregation (being a combining agg of some base table agg) or they reference a base table dimension(s) * Added validation for projection virtual columns to ensure they reference a base table dimension * Tests for rollup projections * Tests for `OnHeapAggregationProjection` validations (cherry picked from commit 30b7bf0c9b17b1087c0ecbac85ecf5aeb3ae99dc) -- 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]
