fixes checkstyle errors

Author: ahmarsuhail

hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/AbstractContractContentSummaryTest.java

@@ -30,41 +30,36 @@
 import static org.apache.hadoop.fs.contract.ContractTestUtils.touch;
 import static org.apache.hadoop.test.LambdaTestUtils.intercept;
 
-public abstract class AbstractContractContentSummaryTest extends AbstractFSContractTestBase  {
+public abstract class AbstractContractContentSummaryTest extends AbstractFSContractTestBase {
 
-    @Test
-    public void testGetContentSummary() throws Throwable {
-        FileSystem fs = getFileSystem();
+  @Test
+  public void testGetContentSummary() throws Throwable {
+    FileSystem fs = getFileSystem();
 
-        Path parent = path("parent");
-        Path nested = path(parent + "/a/b/c");
-        Path filePath = path(nested + "file.txt");
+    Path parent = path("parent");
+    Path nested = path(parent + "/a/b/c");
+    Path filePath = path(nested + "file.txt");
 
-        fs.mkdirs(parent);
-        fs.mkdirs(nested);
-        touch(getFileSystem(), filePath);
+    fs.mkdirs(parent);
+    fs.mkdirs(nested);
+    touch(getFileSystem(), filePath);
 
-        ContentSummary summary = fs.getContentSummary(parent);
+    ContentSummary summary = fs.getContentSummary(parent);
 
-        Assertions.assertThat(summary.getDirectoryCount())
-                .as("Summary " + summary)
-                .isEqualTo(4);
+    Assertions.assertThat(summary.getDirectoryCount()).as("Summary " + summary).isEqualTo(4);
 
-        Assertions.assertThat(summary.getFileCount())
-                .as("Summary " + summary)
-                .isEqualTo(1);
-    }
+    Assertions.assertThat(summary.getFileCount()).as("Summary " + summary).isEqualTo(1);
+  }
 
-    @Test
-    public void testGetContentSummaryIncorrectPath() throws Throwable {
-        FileSystem fs = getFileSystem();
+  @Test
+  public void testGetContentSummaryIncorrectPath() throws Throwable {
+    FileSystem fs = getFileSystem();
 
-        Path parent = path("parent");
-        Path nested = path(parent + "/a");
+    Path parent = path("parent");
+    Path nested = path(parent + "/a");
 
-        fs.mkdirs(parent);
+    fs.mkdirs(parent);
 
-        intercept(FileNotFoundException.class,
-            () -> fs.getContentSummary(nested));
-    }
+    intercept(FileNotFoundException.class, () -> fs.getContentSummary(nested));
+  }
 }

hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/localfs/TestLocalFSContractContentSummary.java

@@ -24,8 +24,8 @@
 
 public class TestLocalFSContractContentSummary extends AbstractContractContentSummaryTest {
 
-    @Override
-    protected AbstractFSContract createContract(Configuration conf) {
-        return new LocalFSContract(conf);
-    }
+  @Override
+  protected AbstractFSContract createContract(Configuration conf) {
+    return new LocalFSContract(conf);
+  }
 }

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

@@ -3284,8 +3284,8 @@ public S3AFileStatus probePathStatus(final Path path,
     }
 
     @Override
-    public RemoteIterator<S3ALocatedFileStatus> listFilesIterator(final Path path, final boolean recursive)
-            throws IOException {
+    public RemoteIterator<S3ALocatedFileStatus> listFilesIterator(final Path path,
+        final boolean recursive) throws IOException {
       return S3AFileSystem.this.innerListFiles(path, recursive, Listing.ACCEPT_ALL_BUT_S3N, null);
     }
   }

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

@@ -66,7 +66,7 @@ public S3ObjectAttributes(
   /**
    * Construct from the result of a copy and those parameters
    * which aren't included in an AWS SDK response.
-   * @param path
+   * @param path path
    * @param copyResult copy result.
    * @param serverSideEncryptionAlgorithm current encryption algorithm
    * @param serverSideEncryptionKey any server side encryption key?

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

@@ -167,19 +167,21 @@ public ContentSummary getDirSummary(Path dir) throws IOException {
   }
 
   /***
-   * This method builds the set of all directories found under the base path. We need to do this because if the
-   * directory structure /a/b/c was created with a single mkdirs() call, it is stored as 1 object in S3 and the list
-   * files iterator will only return a single entry /a/b/c.
+   * This method builds the set of all directories found under the base path. We need to do this
+   * because if the directory structure /a/b/c was created with a single mkdirs() call, it is
+   * stored as 1 object in S3 and the list files iterator will only return a single entry /a/b/c.
    *
-   * We keep track of paths traversed so far to prevent duplication of work. For eg, if we had a/b/c/file-1.txt and
-   * /a/b/c/file-2.txt, we will only recurse over the complete path once and won't have to do anything for file-2.txt.
+   * We keep track of paths traversed so far to prevent duplication of work. For eg, if we had
+   * a/b/c/file-1.txt and /a/b/c/file-2.txt, we will only recurse over the complete path once
+   * and won't have to do anything for file-2.txt.
    *
    * @param dirSet Set of all directories found in the path
    * @param pathsTraversed Set of all paths traversed so far
    * @param basePath Path of directory to scan
    * @param parentPath Parent path of the current file/directory in the iterator
    */
-  private void buildDirectorySet(Set<Path> dirSet, Set<Path> pathsTraversed, Path basePath, Path parentPath) {
+  private void buildDirectorySet(Set<Path> dirSet, Set<Path> pathsTraversed, Path basePath,
+      Path parentPath) {
 
     if (parentPath == null || pathsTraversed.contains(parentPath) || parentPath.equals(basePath)) {
       return;
@@ -225,7 +227,7 @@ public interface GetContentSummaryCallbacks {
     S3AFileStatus probePathStatus(Path path, Set<StatusProbeEnum> probes) throws IOException;
 
     /***
-     * List all entries under a path
+     * List all entries under a path.
      *
      * @param path
      * @param recursive if the subdirectories need to be traversed recursively