mertak-synnada commented on PR #14754: URL: https://github.com/apache/datafusion/pull/14754#issuecomment-2668295526
While refactoring old sources into the `DataSourceExec`, I intended to centralize this file repartition logic for all file types but didn't consider a custom file type in that case. This looks like a better approach for extensibility 👍 Documenting why the function is in `FileSource` may also help future devs. Thanks 🎉 -- 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
