mehakmeet commented on a change in pull request #1991:
URL: https://github.com/apache/hadoop/pull/1991#discussion_r427058232



##########
File path: 
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAbfsStatistics.java
##########
@@ -133,59 +159,48 @@ public void testDeleteStatistics() throws IOException {
         + "in Abfs");
     AzureBlobFileSystem fs = getFileSystem();
     /*
-     This directory path needs to be root for triggering the
-     directories_deleted counter.
+    This directory path needs to be root for triggering the
+    directories_deleted counter.
      */
     Path createDirectoryPath = path("/");
     Path createFilePath = path(getMethodName());
 
-    Map<String, Long> metricMap = fs.getInstrumentation().toMap();
-    //Test for initial statistic values before deleting any directory or files

Review comment:
       This is outdated code, I have moved all initial values into a single 
method.




----------------------------------------------------------------
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.

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



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

Reply via email to