[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-25 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-868690774 There seems to be no complaint from Yetus now :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-24 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-867921545 I have re-run all the `hadoop-aws` tests, only 2 failure tests so far, turns out to be an IAM configuration issue on my end for the rest of error. I explicitly allow `osm-pds`

[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-23 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-866982708 Here are the failing tests: - org.apache.hadoop.tools.contract.AbstractContractDistCpTest#testDistCpWithIterator `org.junit.runners.model.TestTimedOutException: test

[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-22 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-865863147 @steveloughran I got an error that requires me to provide DynamoDB table to run the S3A test, is this expected? Is there a way to exclude this tests ? -- This is an

[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-22 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-865719554 Agreed on adding slight delay on second retry won't do any harm, I have updated the main code to only just throwing the exception without having any manual try. I'm

[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-22 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-864940353 Hi @bogthe thanks for the feedback. Sorry for getting back on this late, I'm running the test right now and will get back on this soon. And for the testing part:

[GitHub] [hadoop] majdyz commented on pull request #3109: HADOOP-17764. S3AInputStream read does not re-open the input stream on the second read retry attempt

2021-06-21 Thread GitBox
majdyz commented on pull request #3109: URL: https://github.com/apache/hadoop/pull/3109#issuecomment-864940353 Hi @bogthe thanks for the feedback. Sorry for getting back on this late, I'm running the test right now and will get back on this soon. And for the testing part: