Rachelint commented on issue #22710: URL: https://github.com/apache/datafusion/issues/22710#issuecomment-5283031363
> Is this considered complete enough so we can start discussing part 2 of this plan: [#15591 (comment)](https://github.com/apache/datafusion/pull/15591#issuecomment-4742353436) > > > . Initial PR for blocked states: The major issue is to agree on the API changes for GroupValues and GroupsAccumulator, and how to organize future works. Looks like the refactor has stabilized? I’ve been a bit busy with the new job lately, so I haven’t been following this epic too closely. I plan to adapt the #15591 to the new path this weekend and start the review. The main blocked issues : performance and not general enough abstraction have been solved for me. And after that，I think we can create issues about implementing the blocked version group values and accumulators incrementaly. -- 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]
