alamb commented on issue #10782: URL: https://github.com/apache/datafusion/issues/10782#issuecomment-2231748616
> These 3 has no circular dependencies which mean we can/should move them out of core before moving catalog-common out of core. > PhysicalOptimizer I agree -- I think we can pull the physical optimzier rules into their own crate. The challenge would be any back dependencies (e.g. if any depend directly on execution plans in the core like `ParquetExec`. > RuntimeEnv I think this is already out of core (in `datafusion-execution`: https://github.com/apache/datafusion/blob/main/datafusion/execution/src/runtime_env.rs > SessionConfig Likewise, this is already in `datafusion-execution` and not in core: https://github.com/apache/datafusion/blob/main/datafusion/execution/src/config.rs#L94-L93 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
