AdamGS commented on issue #14444: URL: https://github.com/apache/datafusion/issues/14444#issuecomment-2688588184
Seems like that creates a dependency problem I'm not sure how to untangle. `catalog::Session` will need `datasource::FiltFormatFactory`, which needs `FileFormat`, which needs `FileSource`, which needs `FileScanConfig`. I initially considered even returning `Any` instead of an `Ard<dyn FileFormatFactoy>` but many usages don't have a statically known format, and returning a `FileFormat` won't make a difference. I can merge `datafusion-datasource` into `datafusion-catalog`, but also open to other ideas, and I'll also keep at it later today. Hopefully something will come up that won't require a huge breaking change. -- 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
