ahmarsuhail commented on code in PR #6479:
URL: https://github.com/apache/hadoop/pull/6479#discussion_r1462127135
##########
hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/DefaultS3ClientFactory.java:
##########
@@ -320,6 +321,11 @@ private <BuilderT extends S3BaseClientBuilder<BuilderT,
ClientT>, ClientT> void
origin = "SDK region chain";
}
+ if (endpointStr != null && endpointStr.endsWith(CENTRAL_ENDPOINT)) {
Review Comment:
yeah, I saw issues..403's with the landstat buckets iirc. Could be related
to
[this](https://github.com/aws/aws-sdk-java/issues/1338#issuecomment-335841213).
I don't know if those issues still exist, it could be worth testing with
US_EAST_1 to see. It shouldn't make a difference what we set though, if you
haven't got the correct region SDK will make a call to figure out the correct
region and then cache it
--
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]
