[GitHub] [hadoop] ahmarsuhail commented on pull request #4706: HADOOP-18073. Adds in s3 client config and updates list.

2022-09-27 Thread GitBox
ahmarsuhail commented on PR #4706: URL: https://github.com/apache/hadoop/pull/4706#issuecomment-1259367384 closing this, these changes will be part of a new PR I'll open once the feature branch is in sync with trunk. -- This is an automated message from the Apache Git Service. To

[GitHub] [hadoop] ahmarsuhail commented on pull request #4706: HADOOP-18073. Adds in s3 client config and updates list.

2022-08-25 Thread GitBox
ahmarsuhail commented on PR #4706: URL: https://github.com/apache/hadoop/pull/4706#issuecomment-1227258037 @steveloughran / @mukund-thakur , I've opened a PR with the first set of changes for the SDK upgrade, could you please take a look? -- This is an automated message from the Apache

[GitHub] [hadoop] ahmarsuhail commented on pull request #4706: HADOOP-18073. Adds in s3 client config and updates list.

2022-08-25 Thread GitBox
ahmarsuhail commented on PR #4706: URL: https://github.com/apache/hadoop/pull/4706#issuecomment-1227250089 Thanks @dannycjones, have moved those things to constants. RE using `eu-west-1` and the region logic in general: Not using `us-east-1` as `headBucket()` fails with that