2010YOUY01 commented on PR #23274: URL: https://github.com/apache/datafusion/pull/23274#issuecomment-4853561480
I think this design should be implemented as the first PR for #7065 🤔 . It does not seem possible to introduce an intermediate step like this PR that only splits out the `EmitTo` part, since it is tightly coupled with the execution logic and the accumulator physical layout. I agree that the initial blocked-state PR may become large, so we will need to figure out how to arrange and split the work better. For coordination, the initial blocked-state PR would depend on https://github.com/apache/datafusion/issues/22710 being finished first. -- 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]
