Skip to content

fix: Preserve nullability in CTE recursive schema#22552

Draft
neilconway wants to merge 2 commits into
apache:mainfrom
neilconway:neilc/fix-recursive-cte-explicit-schema
Draft

fix: Preserve nullability in CTE recursive schema#22552
neilconway wants to merge 2 commits into
apache:mainfrom
neilconway:neilc/fix-recursive-cte-explicit-schema

Conversation

@neilconway
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions Bot added sql SQL Planner logical-expr Logical plan and expressions core Core DataFusion crate sqllogictest SQL Logic Tests (.slt) catalog Related to the catalog crate proto Related to proto crate physical-plan Changes to the physical-plan crate labels May 27, 2026
@neilconway
Copy link
Copy Markdown
Contributor Author

cc @kosiew I'll fill in the rest of the PR template tomorrow, but lmk what you think of this approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

catalog Related to the catalog crate core Core DataFusion crate logical-expr Logical plan and expressions physical-plan Changes to the physical-plan crate proto Related to proto crate sql SQL Planner sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recursive CTE Nullability Handling Should Preserve Logical Schema Without Requiring SQL Rewrites

1 participant