AnthonyZhOon opened a new pull request, #12624: URL: https://github.com/apache/datafusion/pull/12624
## Which issue does this PR close? Part of #12555 ## Rationale for this change Supporting [SessionStateBuilder](https://docs.rs/datafusion/latest/datafusion/execution/session_state/struct.SessionStateBuilder.html) to implement Debug <!-- Why are you proposing this change? If this is already explained clearly in the issue then this section is not needed. Explaining clearly why changes are proposed helps reviewers understand your changes and offer better suggestions for fixes. --> ## What changes are included in this PR? 1. Require `Debug` for PhysicalOptimizerRule 2. `#derive` Debug where necessary 3. Add a `&` to take a `&JoinType` in `EnforceDistribution` (not sure why this wasn't a `cargo check` error but rust-analyzer showed it and types seem to require it) ## Are these changes tested? By CI and the compiler ## Are there any user-facing changes? API change for `PhysicalOptimizerRule` which now requires Debug -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
