Clean up comments.

smengcl · Aug 10, 2023 · e61f564 · e61f564
1 parent 2d6f507
commit e61f564
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/storage/BlockLocationInfo.java b/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/storage/BlockLocationInfo.java
@@ -164,7 +164,7 @@ public BlockLocationInfo build() {
   }
 
   @Override
-  public String  toString() {
+  public String toString() {
     return "{blockID={" + blockID + "}" +
         ", length=" + length +
         ", offset=" + offset +

diff --git a/...zone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OMKeyRequest.java b/...zone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OMKeyRequest.java
@@ -877,10 +877,10 @@ protected void filterOutBlocksStillInUse(OmKeyInfo referenceKey,
             omKeyLocationInfo.getBlockID().getContainerBlockID())
         .collect(Collectors.toCollection(HashSet::new));
 
-    // Use iterator instead of for/forEach for ease of entry removal
+    // Using iterator instead of for/forEach for ease of in-place entry removal
     Iterator<OmKeyInfo> iterOmKeyInfo = keysToBeFiltered
         .getOmKeyInfoList().iterator();
-    // Pardon the loops. ContainerBlockID is nested 9-layers deep:
+    // Pardon the loops. ContainerBlockID is nested 9-layer deep:
     // keysToBeFiltered
     //     .getOmKeyInfoList()
     //     .get(0)
@@ -926,19 +926,19 @@ protected void filterOutBlocksStillInUse(OmKeyInfo referenceKey,
             }
           }
 
-          // Optional cleanup
+          // Cleanup
           if (omKeyLocationInfoList.isEmpty()) {
             itMap.remove();
           }
         }
 
-        // Optional cleanup
+        // Cleanup
         if (keyLocInfoGroup.getLocationVersionMap().isEmpty()) {
           iterKeyLocInfoGroup.remove();
         }
       }
 
-      // Optional cleanup
+      // Cleanup
       if (oldOmKeyInfo.getKeyLocationVersions().isEmpty()) {
         iterOmKeyInfo.remove();
       }