HADOOP-18117. fix yetus checkstyle

Author: mohan3d

hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/DistCpOptions.java

@@ -795,8 +795,8 @@ public Builder withUseIterator(boolean useItr) {
       return this;
     }
 
-    public Builder withUpdateRootDirectoryAttributes(boolean updateRootDirectoryAttributes) {
-      this.updateRootDirectoryAttributes = updateRootDirectoryAttributes;
+    public Builder withUpdateRootDirectoryAttributes(boolean updateRootDirectoryAttrs) {
+      this.updateRootDirectoryAttributes = updateRootDirectoryAttrs;
       return this;
     }
   }

hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/mapred/CopyCommitter.java

@@ -343,7 +343,9 @@ private void preserveFileAttributesForDirectories(Configuration conf)
         //
         boolean skipRootDirectoryAttributes =
             syncOrOverwrite && !updateRootDirectoryAttributes;
-        if (targetRoot.equals(targetFile) && skipRootDirectoryAttributes) continue;
+        if (targetRoot.equals(targetFile) && skipRootDirectoryAttributes) {
+          continue;
+        }
 
         FileSystem targetFS = targetFile.getFileSystem(conf);
         DistCpUtils.preserve(targetFS, targetFile, srcFileStatus, attributes,

hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/TestDistCpSystem.java

@@ -569,7 +569,7 @@ public void testUpdateRootDirectoryAttributes() throws Exception {
 
     // Create a source dir
     fs.mkdirs(source);
-    fs.setOwner(source,"userA", "groupA");
+    fs.setOwner(source, "userA", "groupA");
     fs.setTimes(source, new Random().nextLong(), new Random().nextLong());
 
     GenericTestUtils.createFiles(fs, source, 3, 5, 5);