jihoonson edited a comment on pull request #9898:
URL: https://github.com/apache/druid/pull/9898#issuecomment-650675024


   @FrankChen021 thanks for the error. It does seem there is some 
incompatibility between them. We are using aws sdk 1.11.199 which was released 
in 2017. This version of sdk adds a couple of headers and I'm not sure what are 
not supported by OSS. Even though the OSS doc was updated last year, I guess 
the doc doesn't show the compatibility level correctly. This makes me think at 
least there could be some OSS APIs not compatible to s3 APIs.
   
   Looking at the CI failure, some job failures seem legit.
   
   ```
       ../docs/development/extensions-contrib/aliyun-oss-extensions.md
          44 | |`druid.storage.type`| | oss|Must be set.| 
          52 | |`druid.indexer.logs.type`| | oss|Must be set.| 
   ```
   
   The CI failed for doc, you need to add `oss` to the `./website/.spelling` 
file.
   
   The integration test failure also looks legit. Would you please check it?
   
   ```
   2020-06-27T00:11:30,200 ERROR [main] 
org.apache.druid.testing.utils.LoggerListener - Failed 
[org.apache.druid.tests.indexer.ITOssToOssParallelIndexTest.testAliyunOssIndexData]
 with parameters [Pair{lhs=uris, 
rhs=[oss://%%BUCKET%%/%%PATH%%wikipedia_index_data1.json, 
oss://%%BUCKET%%/%%PATH%%wikipedia_index_data2.json, 
oss://%%BUCKET%%/%%PATH%%wikipedia_index_data3.json]}]
   java.lang.RuntimeException: java.lang.NullPointerException
        at 
org.apache.druid.tests.indexer.AbstractOssInputSourceParallelIndexTest.lambda$doTest$0(AbstractOssInputSourceParallelIndexTest.java:117)
 ~[test-classes/:?]
        at 
org.apache.druid.tests.indexer.AbstractITBatchIndexTest.doIndexTest(AbstractITBatchIndexTest.java:117)
 ~[test-classes/:?]
        at 
org.apache.druid.tests.indexer.AbstractOssInputSourceParallelIndexTest.doTest(AbstractOssInputSourceParallelIndexTest.java:121)
 ~[test-classes/:?]
        at 
org.apache.druid.tests.indexer.ITOssToOssParallelIndexTest.testAliyunOssIndexData(ITOssToOssParallelIndexTest.java:47)
 ~[test-classes/:?]
   ```


----------------------------------------------------------------
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]

Reply via email to