alamb commented on issue #14444: URL: https://github.com/apache/datafusion/issues/14444#issuecomment-2694410637
> The PR is out, but I suspect it's too big. I tried to get it in the 46 release, but now that I probably missed that, [@alamb](https://github.com/alamb) (or any other reviewers) would you prefer I split it up a bit? I'll try and check it out today. > 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 was thinking about this one. It seems like `FileFormat` and `FileSource` may now have some non trivial overlap in functionalty. Maybe we need to rethink some of the APIs / abstractions 🤔 -- 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
