sodonnel commented on code in PR #3661:
URL: https://github.com/apache/ozone/pull/3661#discussion_r952433248
##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/ECContainerReplicaCount.java:
##########
@@ -196,6 +196,29 @@ public Set<Integer> maintenanceIndexes() {
return maintenanceIndexes.keySet();
}
+ /**
+ * Gets a set containing all maintenance only indexes. These are replicas
+ * that are only on maintenance nodes, without any copies on healthy nodes.
+ *
+ * @param includePendingAdd true if any indexes that are pending an add
+ * {@link ContainerReplicaOp.PendingOpType#ADD} should be excluded
+ * @return set containing maintenance only indexes or empty set if none are
+ * in maintenance
+ */
+ public Set<Integer> maintenanceOnlyIndexes(boolean includePendingAdd) {
Review Comment:
I think we should remove the other version of maintenanceOnlyIndexes that
returns a list, and make everything use this one. It keeps it consistent with
decommissionOnlyIndexes
--
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]
