berkaysynnada commented on issue #15111: URL: https://github.com/apache/datafusion/issues/15111#issuecomment-2709783541
I've talked with @jayzhan211 about this, and he has a thought like > I think the overall dependency should be like. Catalog->Schema->Table->File->PhysicalPlan, so access datasource from physical plan violate the dependency So, I'd like to share it with all people. If that's indeed a real problem, and we must obey that dependency chain, I'll have another proposal; move DataSinkExec and other related stuff to the datasource crate as well, and rename it as something like "connectors" or anything related with IO. -- 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
