pitrou commented on issue #38309:
URL: https://github.com/apache/arrow/issues/38309#issuecomment-1831631599
I think the idea of `AddFactory` is that it takes a URI.
Though it seems that it should also declare the accepted URI schemes, so
that we don't invoke all factories when parsing an URI.
So it would be something like:
```c++
using FileSystemFactoryFunc = std::function<
Result<std::shared_ptr<FileSystem>>(const URI& uri)>;
Status AddFileSystemFactory(std::vector<std::string> uri_schemes,
FileSystemFactoryFunc);
```
I agree that something also needs to be done for ancillary functionality
such as resolving a S3 region.
--
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]
