HADOOP-19278. yetus feedback

Change-Id: I9f41965bad7d5b9189f67593b067b42cdec5f605

Author: steveloughran

hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java

@@ -4261,9 +4261,8 @@ PutObjectResponse executePut(
       final File file)
       throws IOException {
     String key = putObjectRequest.key();
-    long len = getPutRequestLength(putObjectRequest);
     ProgressableProgressListener listener =
-        new ProgressableProgressListener(store, putObjectRequest.key(), progress);
+        new ProgressableProgressListener(store, key, progress);
     UploadInfo info = putObject(putObjectRequest, file, listener);
     PutObjectResponse result = store.waitForUploadCompletion(key, info).response();
     listener.uploadCompleted(info.getFileUpload());

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/performance/ITestS3ARenameCost.java

@@ -126,7 +126,7 @@ public void testCostOfRootFileRename() throws Throwable {
       verifyMetrics(() -> {
         fs.rename(src, dest);
         return "after fs.rename(/src,/dest) " + getMetricSummary();
-        },
+      },
           always(FILE_STATUS_FILE_PROBE
               .plus(GET_FILE_STATUS_FNFE)
               .plus(COPY_OP)),
@@ -160,7 +160,7 @@ public void testCostOfRootFileDelete() throws Throwable {
       verifyMetrics(() -> {
         fs.delete(src, false);
         return "after fs.delete(/dest) " + getMetricSummary();
-        },
+      },
           with(DIRECTORIES_CREATED, 0),
           with(DIRECTORIES_DELETED, 0),
           with(FAKE_DIRECTORIES_DELETED, 0),

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/tools/ITestMarkerTool.java

@@ -87,7 +87,6 @@ public void testCleanMarkersFileLimit() throws Throwable {
     markerTool(EXIT_INTERRUPTED, getFileSystem(),
         createdPaths.base, false, 0, 1, false);
   }
-  
 
   @Test
   public void testRenameKeepingFS() throws Throwable {
@@ -112,7 +111,7 @@ public void testRenameKeepingFS() throws Throwable {
     LOG.info("Auditing destination paths");
     verifyRenamed(dest, createdPaths);
   }
-  
+
   /**
    * Assert that an expected number of markers were deleted.
    * @param expected expected count.