FrankChen021 commented on pull request #9898: URL: https://github.com/apache/druid/pull/9898#issuecomment-650562183
Hi @jihoonson , s3 extension does not work with aliyun OSS although OSS provides compatibility with s3. I tested to utilize OSS with s3 extension in the past, but the `putObject` returned HTTP 400 error, saying there were some unsupported HEADERS in the request generated by s3 SDK. I didn't explore that problem very deeply, I guess the latest s3 SDK adds some new HEADERS which are not compatible in OSS. the APIs between OSS and s3 are very similar, even their names. but I don't think it's a good idea to use one extension to support both OSS and s3, or use some abstraction method to extract the same code between s3 and OSS. Actually, this kind of abstraction has been done by deep storage framework, so every implementation interacts with its underlying storage API. As s3 and OSS are provided by two companies, it's better to provide two extensions respectively so that they can evolve independently. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
