asolimando commented on PR #24716: URL: https://github.com/apache/datafusion/pull/24716#issuecomment-5449365016
@kosiew a small follow-up of https://github.com/apache/datafusion/pull/23651, this is a pure refactor to properly propagate `PhysicalOptimizerContext`, which is meant to be a convenient way to pass context to physical rules, especially those dealing with CBO, so we can enrich context for statistics propagation without breaking signatures. I have a question: since it's a pure refactor and it doesn't really alter coverage, I would consider https://github.com/apache/datafusion/pull/24716#issuecomment-5436734704, wdyt? Thanks! -- 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]
