alamb opened a new pull request, #10794: URL: https://github.com/apache/datafusion/pull/10794
## Which issue does this PR close? Part of https://github.com/apache/datafusion/issues/10782 ## Rationale for this change the `datafusion/core/src/execution/context/mod.rs` file is quite large and hard to navigate. It would be better to have SessionContext and SessionState in their own modules Also, if we ever are able to break out catalog from the core, e.g. https://github.com/apache/datafusion/issues/10782 we may have to break out `SessionState` as well, so having SessionState in its own module will help here It also turns out splitting them into their own files makes the boundaries a bit clearer ## What changes are included in this PR? 1. Move `SessionState` out of `datafusion/core/src/execution/context/mod.rs` and into `datafusion/core/src/execution/state.rs` 2. Add a `pub use` 3. Add some additional accessors to `SessionState` to maintain the same behavior ## Are these changes tested? Exiting CI ## Are there any user-facing changes? No, this is an internal code reorganization only -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
