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

    https://github.com/apache/storm/pull/2631#discussion_r180755909
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/scheduler/Cluster.java ---
    @@ -79,8 +81,8 @@
         private Set<String> blackListedHosts = new HashSet<>();
         private INimbus inimbus;
         private final Topologies topologies;
    -    private final Map<String, Double> scheduledCpuCache = new HashMap<>();
    -    private final Map<String, Double> scheduledMemoryCache = new 
HashMap<>();
    +    private final Map<String, Map<WorkerSlot, NormalizedResourceRequest>> 
nodeToScheduledResourcesCache;
    +    private final HashMap<String, Set<WorkerSlot>> nodeToUsedPortsCache;
    --- End diff --
    
    use Map instead of HashMap?


---

Reply via email to