alamb opened a new pull request, #5432: URL: https://github.com/apache/arrow-datafusion/pull/5432
# Which issue does this PR close? Part of splitting out physical plan: https://github.com/apache/arrow-datafusion/issues/1754 First part of https://github.com/apache/arrow-datafusion/issues/5405 # Rationale for this change the `datafusion` crate is quite large as it has all the physical plan code as well as the physical optimizer I would like to split out physical_plan and physical_optimizer (to mimic datafusion_expr and datafusion_optimizer). However in order to do so I need to move parts of datafusion that they depend on to a new module # What changes are included in this PR? # Changes: 1. create a new `datafusion-execution` crate 2. Extract some (but not all) of `datafusion/core/src/execution` to that new crate Items remaining (planned as follow on PRs): 1. Figure out how to decouple ListingTableFactory and RuntimeEnv (I have an idea, but will easier to review in follow on PRs) 2. Extract structures from context.rs (specifically SessionState and then the things that it depends on) I plan to leave `SessionContext` in the core datafusion crate as it pulls everything together (relies on everything) # Are these changes tested? Covered by existing tests # Are there any user-facing changes? There will be a new crate, but otherwise no -- 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]
