adoroszlai commented on code in PR #10080:
URL: https://github.com/apache/ozone/pull/10080#discussion_r3090861719


##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/placement/algorithms/SCMContainerPlacementRackAware.java:
##########
@@ -537,7 +537,7 @@ private DatanodeDetails chooseNode(List<DatanodeDetails> 
excludedNodes,
   /**
    * Choose a batch of datanodes on different rack than excludedNodes or
    * chosenNodes.
-   * TODO HDDS-7226: Update Implementation to accomodate for already used
+   * TODO HDDS-7226: Update Implementation to accommodate for already used
    * nodes to conform to existing placement policy.

Review Comment:
   This is leftover TODO
   
   ```suggestion
   ```



##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelinePlacementPolicy.java:
##########
@@ -235,7 +235,7 @@ private boolean 
multipleRacksAvailable(List<DatanodeDetails> dns) {
 
   /**
    * Pipeline placement choose datanodes to join the pipeline.
-   * TODO: HDDS-7227: Update Implementation to accomodate for already used
+   * TODO: HDDS-7227: Update Implementation to accommodate for already used
    * nodes in pipeline to conform to existing placement policy.

Review Comment:
   ```suggestion
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to