This is an automated email from the ASF dual-hosted git repository.
sodonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git
The following commit(s) were added to refs/heads/master by this push:
new 2650723 HDDS-4343. ReplicationManager.handleOverReplicatedContainer()
does not handle unhealthyReplicas properly. (#1495)
2650723 is described below
commit 26507233e4f60f5d123d5bd315aba3c6bb90d570
Author: GlenGeng <gleng...@tencent.com>
AuthorDate: Thu Oct 15 04:13:01 2020 +0800
HDDS-4343. ReplicationManager.handleOverReplicatedContainer() does not
handle unhealthyReplicas properly. (#1495)
---
.../java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java
index a010910..bab8851 100644
---
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java
+++
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java
@@ -746,8 +746,9 @@ public class ReplicationManager
if (excess > 0) {
sendDeleteCommand(container, r.getDatanodeDetails(), true);
excess -= 1;
+ } else {
+ break;
}
- break;
}
// After removing all unhealthy replicas, if the container is still over
// replicated then we need to check if it is already mis-replicated.
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org
