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

[GitBox <gi...@apache.org>]

monthonk commented on code in PR #4140:
URL: https://github.com/apache/hadoop/pull/4140#discussion_r842712497


##########
hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/tools/ITestMarkerTool.java:
##########
@@ -307,22 +309,25 @@ public void testRunLimitedAudit() throws Throwable {
   }
 
   /**
-   * Run an audit against the landsat bucket.
+   * Run an audit against a bucket with a large number of objects.
    * <p></p>
    * This tests paging/scale against a larger bucket without
    * worrying about setup costs.
    */
   @Test
   public void testRunLimitedLandsatAudit() throws Throwable {

Review Comment:
   Should we update the method name? it's still landsat.



-- 
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: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org