Github user jerrypeng commented on a diff in the pull request:
https://github.com/apache/storm/pull/2442#discussion_r157677523
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/multitenant/NodePool.java
---
@@ -98,7 +98,7 @@ public RoundRobinSlotScheduler(TopologyDetails td, int
slotsToUse,
}
_spreadToSchedule = new HashMap<>();
- List<String> spreadComps =
(List<String>)td.getConf().get(Config.TOPOLOGY_SPREAD_COMPONENTS);
+ List<String> spreadComps =
(List<String>)td.getConf().get(Config.TOPOLOGY_RAS_CONSTRAINT_SPREAD_COMPONENTS);
--- End diff --
I don't know about changing TOPOLOGY_SPREAD_COMPONENTS to
TOPOLOGY_RAS_CONSTRAINT_SPREAD_COMPONENTS. It seems kind of weird to me that
the multitenant scheduler would have a config that references RAS.
---
