mbutrovich opened a new pull request, #23579: URL: https://github.com/apache/datafusion/pull/23579
## Which issue does this PR close? - Backport of #22552 to `branch-54` (for 54.1.0, tracked in #22547). - Fixes execution failure #22034 (regression introduced in 54.0.0 by #21912, which made nullability checking strict without fixing schema derivation). ## Rationale for this change Recursive CTE queries with certain nullability patterns fail at execution in 54.0.0. This restores correct schema derivation for recursive queries. Reviewer note: this has a larger blast radius than the other 54.1.0 backports. It touches 12 files, adds a field to `LogicalPlan::RecursiveQuery`, and includes proto changes (`proto/src/logical_plan/mod.rs`). Worth a deliberate call on whether it clears the patch-release stability bar. ## What changes are included in this PR? Clean cherry-pick of #22552. No adaptation required. ## Are these changes tested? Yes. Carries the original regression coverage (`cte.slt`, `explain_tree.slt`, `explain_analyze.rs`), all tests pass. ## Are there any user-facing changes? Recursive CTEs with these nullability patterns execute again. Adds a field to `LogicalPlan::RecursiveQuery` and the corresponding proto messages (internal, but on public-facing structs). -- 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]
