alamb commented on issue #19649: URL: https://github.com/apache/datafusion/issues/19649#issuecomment-3729744429
> I have a somewhat crazy thought when I was reviewing the previous blocked aggregation PR: should we throw away all the existing aggregation code and rewrite it from scratch for this blocked design? TLDR is I think this will be required to actually impement a blocked aggregation design The trick will be how to incrementally roll out such a feature, given the extend of the existing aggregation code. I think this is doable, but will require a significant and concerted effort. @Rachelint also did quite a bit of work on this in PRs like https://github.com/apache/datafusion/pull/15591 so perhaps they have some context to share -- 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]
