bipinprasad commented on a change in pull request #3379: URL: https://github.com/apache/storm/pull/3379#discussion_r581619307
########## File path: storm-server/src/main/java/org/apache/storm/scheduler/ISchedulingState.java ########## @@ -339,4 +354,31 @@ boolean wouldFit( * Get the nimbus configuration. */ Map<String, Object> getConf(); + + /** + * Determine the list of racks on which topologyIds have been assigned. Note that the returned set + * may contain {@link DNSToSwitchMapping#DEFAULT_RACK} if {@link #getHostToRack()} is null or + * does not contain the assigned host. + * + * @param topologyIds for which assignments are examined. + * @return set of racks on which assignments have been made. + */ + default Set<String> getAssignedRacks(String... topologyIds) { Review comment: This will be used in defragmentation code. ---------------------------------------------------------------- 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