berkaysynnada commented on PR #11060: URL: https://github.com/apache/datafusion/pull/11060#issuecomment-2186375631
Thank you, @devinjdangelo, for proposing this extensibility feature. I would like to clarify a few points to ensure I understand correctly: * We cannot merge `FileType` and `FileFormatFactory` due to dependency issues. * We cannot merge `FileFormatFactory` and `FileFormat` because of stateful format factories. If these points are correct, I am considering whether we could move these states (extension options, like `CsvOptions`) into `FileType` somehow to remove one of these layers. This is just a thought experiment; I don't want to block the PR, which I believe improves extensibility. -- 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]
