junkaixue commented on code in PR #2848:
URL: https://github.com/apache/helix/pull/2848#discussion_r1690498610
##########
helix-core/src/main/java/org/apache/helix/controller/dataproviders/BaseControllerDataProvider.java:
##########
@@ -1078,12 +1081,16 @@ private void
updateDisabledInstances(Collection<InstanceConfig> allInstanceConfi
// Move the calculating disabled instances to refresh
_disabledInstanceForPartitionMap.clear();
_disabledInstanceSet.clear();
+ _disabledInstancesForAllPartitionsSet.clear();
for (InstanceConfig config : allInstanceConfigs) {
Map<String, List<String>> disabledPartitionMap =
config.getDisabledPartitionsMap();
- if (config.getInstanceOperation().getOperation()
- .equals(InstanceConstants.InstanceOperation.DISABLE)) {
+ // Treat instance as disabled if it has "DISABLE" operation or
"ALL_RESOURCES" in the disabled partition map
+ if
(config.getInstanceOperation().getOperation().equals(InstanceConstants.InstanceOperation.DISABLE))
{
_disabledInstanceSet.add(config.getInstanceName());
}
+ if
(disabledPartitionMap.containsKey(InstanceConstants.ALL_RESOURCES_DISABLED_PARTITION_KEY))
{
+ _disabledInstancesForAllPartitionsSet.add(config.getInstanceName());
Review Comment:
Not make it in base controller. This resource assignment, make it in
resource controller provider
--
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: reviews-unsubscr...@helix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@helix.apache.org
For additional commands, e-mail: reviews-h...@helix.apache.org
