[GitHub] [hadoop] sabertiger commented on pull request #5024: HADOOP-18233. Possible race condition with TemporaryAWSCredentialsPro…

2022-10-31 Thread GitBox
sabertiger commented on PR #5024: URL: https://github.com/apache/hadoop/pull/5024#issuecomment-1297632934 @steveloughran same user on ASF Jira: sabertiger -- 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

[GitHub] [hadoop] sabertiger commented on pull request #5024: HADOOP-18233. Possible race condition with TemporaryAWSCredentialsPro…

2022-10-25 Thread GitBox
sabertiger commented on PR #5024: URL: https://github.com/apache/hadoop/pull/5024#issuecomment-1290897482 > i really like the tests; you've gone to a lot of effort to show how the code is broken. > > i do think we can simplify the production code though. > > can't we just make

[GitHub] [hadoop] sabertiger commented on pull request #5024: HADOOP-18233. Possible race condition with TemporaryAWSCredentialsPro…

2022-10-18 Thread GitBox
sabertiger commented on PR #5024: URL: https://github.com/apache/hadoop/pull/5024#issuecomment-1282812020 @ashutoshcipher Trying to gauge interest on PR for a deprecated component. Due to deprecation, some of the automated checks are failing. The Yetus/checkstyle errors are

[GitHub] [hadoop] sabertiger commented on pull request #5024: HADOOP-18233. Possible race condition with TemporaryAWSCredentialsPro…

2022-10-17 Thread GitBox
sabertiger commented on PR #5024: URL: https://github.com/apache/hadoop/pull/5024#issuecomment-1281384999 Integration test results for s3a/s3n @ us-west-2: `# mvn clean verify ` [INFO] --- maven-failsafe-plugin:3.0.0-M1:integration-test (default) @ hadoop-aws --- [INFO]