fix checkstyle

Author: huangzhaobo99

hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSDirRenameOp.java

@@ -85,8 +85,8 @@ private static Triple<Boolean, Optional<QuotaCounts>, Optional<QuotaCounts>> ver
     // Verify path without valid 'DirectoryWithQuotaFeature'
     // Note: In overwrite scenarios, quota calculation is still required,
     // overwrite operations delete existing content, which affects the root directory's quota.
-    if (!overwrite && FSDirectory.verifyPathWithoutValidFeature(src)
-        && FSDirectory.verifyPathWithoutValidFeature(dst)) {
+    if (!overwrite && FSDirectory.verifyPathWithoutValidQuotaFeature(src)
+        && FSDirectory.verifyPathWithoutValidQuotaFeature(dst)) {
       return Triple.of(false, srcDelta, dstDelta);
     }
 

hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSDirectory.java

@@ -1244,13 +1244,12 @@ static void verifyQuota(INodesInPath iip, int pos, QuotaCounts deltas,
 
   /**
    * Verify that the path from the specified position to the root
-   * (excluding the root itself) does not contain any valid quota features.
+   * (excluding the root itself) does not contain any valid quota feature.
    *
-   * @param iip the INodesInPath containing all the ancestral INodes.
-   * @return true if no valid features are found along the path,
-   * false if any directory in the path has an active feature.
+   * @param iip the INodesInPath instance containing all the INodes for the path.
+   * @return true if no valid quota feature, false if any directory in the path has quota feature.
    */
-  static boolean verifyPathWithoutValidFeature(INodesInPath iip) {
+  static boolean verifyPathWithoutValidQuotaFeature(INodesInPath iip) {
     for (int i = iip.length() - 2; i >= 1; i--) {
       INodeDirectory d = iip.getINode(i).asDirectory();
       if (d.isWithQuota()) {

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestCorrectnessOfQuotaAfterRenameOp.java

@@ -257,15 +257,15 @@ public void testRenameUndoWithoutValidFeature() throws Exception {
       Path file1 = new Path(srcDir3, "file" + i);
       DFSTestUtil.createFile(dfs, file1, fileLen, replication, 0);
     }
-    
+
     // Test rename1
     ContentSummary rootContentSummary1 = dfs.getContentSummary(new Path("/"));
     QuotaUsage rootQuotaUsage1 = dfs.getQuotaUsage(new Path("/"));
     ContentSummary contentSummary1 = dfs.getContentSummary(testParentDir1);
-    
+
     FSNamesystem fsn = cluster.getNamesystem();
     FSDirectory fsDirectory = fsn.getFSDirectory();
-    
+
     // Replace INode, expected addChild return false
     INodeDirectory dir = fsDirectory.getINode4Write(testParentDir2.toString()).asDirectory();
     INodeDirectory mockDir = Mockito.spy(dir);