walterddr opened a new issue, #11034: URL: https://github.com/apache/pinot/issues/11034
Background === - this is a follow-up on #10745 - original design doc: https://docs.google.com/document/d/1Us6aBvTpNLMEy0ODo34OgTk73h_LVFFAH6q17689h1M/edit?usp=sharing Summary === The previous iteration of aggregate function refactor allows all pinot v1 aggregate functions to be planned with v2 engine by registering aggregate functions with v1's agg-function type into multiple aggregate function signatures on v2. This is a follow-up to ensure the plan can be broken down with the variety of aggregate function flavors, specifically 1. support aggregate function with dynamic return type 2. support aggregate function with the literal positional argument used during function constructor Furthermore, we want to clean up some of the components we had to introduce during the 2 initial PR, e.g. 1. cleaner function registry mechanism without specific calcite dependency (they are relatively hard to understand) 2. clean up hint-based aggregate node representation (currently very complex and not used the same system between runtime and planning) 3. clean up the final-stage agg vs. final-stage project (so that we don't have to differentiate; and can merge with potential project) -- 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]
