Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Move LogicalPlan to datafusion-expr crate #2286

Closed
wants to merge 4 commits into from

Conversation

andygrove
Copy link
Member

Which issue does this PR close?

Follows on from #2284

Closes #2245

Rationale for this change

Move LogicalPlan to datafusion-expr crate so that we can refer to LogicalPlan from Expr, which is needed for subquery support.

What changes are included in this PR?

Moved a bunch of files from core crate to expr crate.

Are there any user-facing changes?

Yes, API changes.

@github-actions github-actions bot added ballista datafusion Changes in the datafusion crate labels Apr 20, 2022
@andygrove andygrove closed this Apr 20, 2022
@andygrove andygrove deleted the move-logical-plan-1 branch January 27, 2023 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
datafusion Changes in the datafusion crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move LogicalPlan to datafusion-expr crate
1 participant