sodonnel commented on a change in pull request #1495:
URL: https://github.com/apache/hadoop-ozone/pull/1495#discussion_r504569348
##########
File path:
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java
##########
@@ -746,8 +746,9 @@ private void handleOverReplicatedContainer(final
ContainerInfo container,
if (excess > 0) {
sendDeleteCommand(container, r.getDatanodeDetails(), true);
excess -= 1;
+ } else {
+ break;
}
- break;
Review comment:
@linyiqun What if there is an excess of containers, but all containers
are somehow unhealthy? We don't want RM to remove all copies of them, as that
could result in a lost container. That is why we limit the number of unhealthy
containers removed to the excess number.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org
