alamb commented on code in PR #26479: URL: https://github.com/apache/datafusion/pull/26479#discussion_r4306988233
########## datafusion/physical-optimizer/src/ensure_requirements/mod.rs: ########## @@ -18,9 +18,13 @@ //! [`EnsureRequirements`] optimizer rule that enforces distribution and //! sorting requirements together so that the two never invalidate each other. //! -//! This rule replaces the separate `EnforceDistribution` + `EnforceSorting` Review Comment: I also removed comments about how the code used to be structured as I don't think that is relevant anymore -- 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]
