Skip to content

Commit 7a2162c

Browse files
committed
fix checkstyle
1 parent 3b3b34e commit 7a2162c

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ private static Triple<Boolean, Optional<QuotaCounts>, Optional<QuotaCounts>> ver
8383
}
8484

8585
// Verify path without valid 'DirectoryWithQuotaFeature'
86-
// Note: In overwrite scenarios, quota calculation is still required,
86+
// Note: In overwrite scenarios, quota calculation is still required,
8787
// overwrite operations delete existing content, which affects the root directory's quota.
8888
if (!overwrite && FSDirectory.verifyPathWithoutValidFeature(src)
8989
&& FSDirectory.verifyPathWithoutValidFeature(dst)) {

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
*/
1818
package org.apache.hadoop.hdfs.server.namenode;
1919

20-
import org.apache.hadoop.hdfs.server.namenode.snapshot.DirectorySnapshottableFeature;
2120
import org.apache.hadoop.hdfs.server.namenode.snapshot.Snapshot;
2221
import org.apache.hadoop.util.StringUtils;
2322

@@ -1244,8 +1243,8 @@ static void verifyQuota(INodesInPath iip, int pos, QuotaCounts deltas,
12441243
}
12451244

12461245
/**
1247-
* Verifies that the path from the specified position to the root
1248-
* does not contain any valid features.
1246+
* Verify that the path from the specified position to the root
1247+
* (excluding the root itself) does not contain any valid quota features.
12491248
*
12501249
* @param iip the INodesInPath containing all the ancestral INodes.
12511250
* @return true if no valid features are found along the path,
@@ -1362,6 +1361,7 @@ private void copyINodeDefaultAcl(INode child, FsPermission modes) {
13621361
* @param inode the new INode to add
13631362
* @param modes create modes
13641363
* @param checkQuota whether to check quota
1364+
* @param updateQuota whether to update quota
13651365
* @return an INodesInPath instance containing the new INode
13661366
*/
13671367
@VisibleForTesting

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ public void testQuotaUsageWhenRenameWithSameStoragePolicy() throws Exception {
8080
DFSTestUtil.createFile(dfs, file2, fileLen, replication, 0);
8181

8282
final Path dstDir1 = new Path(testParentDir2, "dst-dir");
83-
// If dstDir1 not exist, after the rename operation,
83+
// If dstDir1 not exist, after the rename operation,
8484
// the root dir's quota usage should remain unchanged.
8585
QuotaUsage quotaUsage1 = dfs.getQuotaUsage(new Path("/"));
8686
ContentSummary cs1 = dfs.getContentSummary(testParentDir1);

0 commit comments

Comments
 (0)