adriangb commented on issue #19351: URL: https://github.com/apache/datafusion/issues/19351#issuecomment-3853827744
I agree that it would be great to have a clear split between a physical execution plan tree and the execution state. I also agree that calling `execute()` essentially produces an implicit tree of execution state via streams, but I don't think the API is good or clear, and e.g. the interaction with dynamic filters is poor. I'm not sure what the best way to solve this is. FWIW I will say that if we were to add a new `execute()` method or change the signature we should make an `ExecuteArgs` struct so that we can add fields in the future and such without breaking it again. -- 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]
