Fixing checkstyle issues

Author: Ritesh Garg

hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java

@@ -648,13 +648,13 @@ protected void setPipeline(LocatedBlock lb) {
     setPipeline(lb.getLocations(), lb.getStorageTypes(), lb.getStorageIDs());
   }
 
-  protected void setPipeline(DatanodeInfo[] nodes, StorageType[] storageTypes,
-                           String[] storageIDs) {
+  protected void setPipeline(DatanodeInfo[] newNodes, StorageType[] newStorageTypes,
+                           String[] newStorageIDs) {
     synchronized (nodesLock) {
-      this.nodes = nodes;
+      this.nodes = newNodes;
     }
-    this.storageTypes = storageTypes;
-    this.storageIDs = storageIDs;
+    this.storageTypes = newStorageTypes;
+    this.storageIDs = newStorageIDs;
   }
 
   /**

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDistributedFileSystem.java

@@ -2680,7 +2680,7 @@ public void testSingleRackFailureDuringPipelineSetupMinReplicationPossible() thr
 
   @Test
   public void testSingleRackFailureDuringPipelineSetupMinReplicationImpossible()
-    throws Exception {
+      throws Exception {
     Configuration conf = getTestConfiguration();
     conf.setClass(DFSConfigKeys.DFS_BLOCK_REPLICATOR_CLASSNAME_KEY,
         BlockPlacementPolicyRackFaultTolerant.class, BlockPlacementPolicy.class);
@@ -2730,7 +2730,8 @@ public void testMultipleRackFailureDuringPipelineSetupMinReplicationPossible() t
   public void testMultipleRackFailureDuringPipelineSetupMinReplicationImpossible()
       throws Exception {
     Configuration conf = getTestConfiguration();
-    conf.setClass(DFSConfigKeys.DFS_BLOCK_REPLICATOR_CLASSNAME_KEY, BlockPlacementPolicyRackFaultTolerant.class,
+    conf.setClass(DFSConfigKeys.DFS_BLOCK_REPLICATOR_CLASSNAME_KEY,
+        BlockPlacementPolicyRackFaultTolerant.class,
         BlockPlacementPolicy.class);
     conf.setBoolean(
         HdfsClientConfigKeys.BlockWrite.ReplaceDatanodeOnFailure.ENABLE_KEY,
@@ -2772,7 +2773,7 @@ public void testAllRackFailureDuringPipelineSetup() throws Exception {
       // create a file with replication 3, for rack fault tolerant BPP,
       // it should allocate nodes in all 3 rack but fail because no DNs are present.
       LambdaTestUtils.intercept(IOException.class,
-        () ->
+          () ->
           DFSTestUtil.createFile(fs, new Path("/testFile"),
               1024L, (short) 3, 1024L));
     }