alamb opened a new pull request, #22445: URL: https://github.com/apache/datafusion/pull/22445
- Backports https://github.com/apache/datafusion/pull/22437 from @alamb to the branch-54 line This PR cherry-picks the revert of `ExecutionPlan::apply_expressions()` (#20337) onto `branch-54` so that DataFusion 54.0 does not ship the new public API. ## Conflict resolution Cherry-pick conflicted in `datafusion/proto/tests/cases/roundtrip_physical_plan.rs`. The conflict was around the `CustomExecWithExprs` test helper, which only exists on `main` (added by a later PR not in `branch-54`). Resolution: kept `HEAD` (`branch-54`) state — i.e. did not introduce the new test helper, since there is nothing to revert there on the release branch. ## Verification Run on the backport branch: - `nice ./ci/scripts/rust_fmt.sh` — passed - `nice ./ci/scripts/rust_clippy.sh` — passed (no warnings) - `nice cargo test --profile=ci --test sqllogictests` — passed (472/472 files) - `nice cargo nextest run` — passed (9154 tests, 7 skipped) -- 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]
