cisaacson commented on issue #11420: URL: https://github.com/apache/datafusion/issues/11420#issuecomment-2226398313
@alamb I agree, removing it or using `Any` would make lots of things challenging. The idea of state for a SessionContext that is accessible from a variety of places is important. The easiest way to address this is to move `SessionState` to a smaller sub-crate like `common-runtime` or even `common-traits` (although the latter would be complicated as many traits depend on structs and other types). I see why it is used in so many places, and I do support the idea that it is important to do so. Perhaps we could just identify all truly common types like this and move them into a sub-crate, that could be done without too much effort. Such a change so incorporate other types of common objects too. In this way anytime someone wants to break something out of `core` the most likely common dependencies would be available in the smaller sub-crate. -- 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
