milenkovicm commented on issue #23194: URL: https://github.com/apache/datafusion/issues/23194#issuecomment-4874870525
Datafusion's first concern, rightly, was not AQE; thus, trying to add one at this moment may be more of a hack rather than a systematic approach to it. I agree with @Dandandan comment https://github.com/apache/datafusion/issues/23194#issuecomment-4859362533 making a plan-observation/runtime-adaptation framework could be a step in right direction, and would help to make some separation of concerns between operators and planner(s). IMHO, making a case how core datafusion would benefit from this would be a good first step. I'd suggest we focus on requirements/gaps rather than prescribing solutions. Also, it would be great if we refrain from speculative comparisons and claims, as they do not really help. -- 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]
