[GitHub] [hadoop] dannycjones commented on a diff in pull request #4140: HADOOP-18168. Fix S3A ITestMarkerTool dependency on purged public bucket

2022-04-29 Thread GitBox
dannycjones commented on code in PR #4140: URL: https://github.com/apache/hadoop/pull/4140#discussion_r861581105 ## hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3ARequesterPays.java: ## @@ -102,14 +88,30 @@ public void testRequesterPaysDisabledFails()

[GitHub] [hadoop] dannycjones commented on a diff in pull request #4140: HADOOP-18168. Fix S3A ITestMarkerTool dependency on purged public bucket

2022-04-26 Thread GitBox
dannycjones commented on code in PR #4140: URL: https://github.com/apache/hadoop/pull/4140#discussion_r858847636 ## hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/S3ATestConstants.java: ## @@ -108,6 +108,21 @@ */ String DEFAULT_REQUESTER_PAYS_FILE =

[GitHub] [hadoop] dannycjones commented on a diff in pull request #4140: HADOOP-18168. Fix S3A ITestMarkerTool dependency on purged public bucket

2022-04-06 Thread GitBox
dannycjones commented on code in PR #4140: URL: https://github.com/apache/hadoop/pull/4140#discussion_r843118842 ## hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/S3ATestConstants.java: ## @@ -108,6 +108,21 @@ */ String DEFAULT_REQUESTER_PAYS_FILE =