[GitHub] storm pull request #2630: STORM-3024: Allow for scheduling to happen in the ...

[agresch]

Github user agresch commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2630#discussion_r180805730
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/DefaultResourceAwareStrategy.java
 ---
    @@ -65,6 +65,9 @@ public SchedulingResult schedule(Cluster cluster, 
TopologyDetails td) {
             final List<ObjectResources> sortedNodes = sortAllNodes(td, null, 
favoredNodes, unFavoredNodes);
     
             for (ExecutorDetails exec : orderedExecutors) {
    +            if (!running) {
    --- End diff --
    
    this is just to force a quicker cancellation of scheduling?


---