[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

[revans2]

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

    https://github.com/apache/storm/pull/2385#discussion_r149161951
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/scheduler/Cluster.java ---
    @@ -556,6 +605,9 @@ public boolean wouldFit(
             }
     
             double memoryAdded = afterTotal - currentTotal;
    +        double memoryAvailable = ObjectReader.getDouble(
    +                
resourcesAvailable.getOrDefault(Constants.COMMON_TOTAL_MEMORY_RESOURCE_NAME, 
null), 0.0);
    --- End diff --
    
    Same as above, just make it a get.


---