AryanMishra1789 commented on issue #20396: URL: https://github.com/apache/datafusion/issues/20396#issuecomment-3914822539
Hi @gabotechs I can see why attaching per node metadata directly to ExecutionPlan would be useful for distributed planning. My main concern is that putting arbitrary annotations on the core trait could make the API harder to evolve over time. A possible alternative would be to keep annotations in a separate context structure keyed by plan nodes, which optimizers can consult when needed. That keeps ExecutionPlan minimal while still allowing composable optimizers to share domain-specific metadata. -- 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]
