wgtmac commented on PR #719: URL: https://github.com/apache/iceberg-cpp/pull/719#issuecomment-4825262656
I agree with your concern that it is wrong to deduce the file system scheme based on `warehouse`. I remember that this is just a temporary workaround and should be removed after we introduce a C++ `ResolvingFileIO` or the like. Perhaps now it is the right time to fix this. Maybe we can get started with expanding the S3-compatible object storage support with AWS SDK and report unsupported error for other storage system. Then we can add `ResolvingFileIO` based on this? -- 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]
