rluvaton commented on PR #25246: URL: https://github.com/apache/datafusion/pull/25246#issuecomment-5664483183
The reason I did not split it already so we can discuss it. I created a huge PR so: 1. We can see what every decision will take us and avoid iterating on main building blocks and creating a lot of breaking changes 2. See if this approach performance is even reasonable 3. See if the traits are allowing to implementing existing and company specific very complex aggregation. I'm writing a summary of all my finding and will post it in the issue and I want to discuss it on the weekly meeting. I don't want to merge prs yet that decide on the blocked path without us having it figured out and understanding the implications. -- 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]
