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