HDDS-11843. Improve Some Code.

Author: fanshilun

hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/RatisContainerReplicaCount.java

@@ -286,7 +286,7 @@ public String toString() {
    *
    * When considering inflight operations, it is assumed any operation will
    * fail. However, to consider the worst case and avoid data loss, we always
-   * assume a delete will succeed and and add will fail. In this way, we will
+   * assume a delete will succeed and add will fail. In this way, we will
    * avoid scheduling too many deletes which could result in dataloss.
    *
    * Decisions around over-replication are made only on healthy replicas,
@@ -320,7 +320,7 @@ public String toString() {
    * For under replicated containers we do consider inflight add and delete to
    * avoid scheduling more adds than needed. There is additional logic around
    * containers with maintenance replica to ensure minHealthyForMaintenance
-   * replia are maintained.
+   * replica are maintained.
    *
    * @return Delta of replicas needed. Negative indicates over replication and
    *         containers should be removed. Positive indicates over replication
@@ -621,7 +621,7 @@ public int getExcessRedundancy(boolean includePendingDelete) {
    * considering inflight add and deletes.
    * @return zero if perfectly replicated, a negative value for over replication
    *         and a positive value for under replication. The magnitude of the
-   *         return value indicates how many replias the container is over or
+   *         return value indicates how many replies the container is over or
    *         under replicated by.
    */
   private int redundancyDelta(boolean includePendingDelete,
@@ -650,7 +650,7 @@ boolean insufficientDueToOutOfService() {
 
   /**
    * How many more replicas can be lost before the container is
-   * unreadable, assuming any infligh deletes will complete. For containers
+   * unreadable, assuming any inflight deletes will complete. For containers
    * which are under-replicated due to decommission
    * or maintenance only, the remaining redundancy will include those
    * decommissioning or maintenance replicas, as they are technically still

hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/RatisUnderReplicationHandler.java

@@ -145,7 +145,7 @@ public int processAndSendCommands(
 
     if (targetDatanodes.size() < replicaCount.additionalReplicaNeeded()) {
       // The placement policy failed to find enough targets to satisfy fix
-      // the under replication. There fore even though some commands were sent,
+      // the under replication. Therefore even though some commands were sent,
       // we throw an exception to indicate that the container is still under
       // replicated and should be re-queued for another attempt later.
       LOG.debug("Placement policy failed to find enough targets to satisfy " +

hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/UnhealthyReplicationProcessor.java

@@ -92,7 +92,7 @@ public void processAll(ReplicationQueue queue) {
     List<HealthResult> failedOnes = new LinkedList<>();
     // Getting the limit requires iterating over all nodes registered in
     // NodeManager and counting the healthy ones. This is somewhat expensive
-    // so we get get the count once per iteration as it should not change too
+    // so we get the count once per iteration as it should not change too
     // often.
     long inflightLimit = replicationManager.getReplicationInFlightLimit();
     while (true) {