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?
---