Github user danny0405 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2647#discussion_r186665919
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/Cluster.java ---
@@ -763,6 +773,7 @@ public void setAssignments(
assertValidTopologyForModification(assignment.getTopologyId());
}
assignments.clear();
+ totalResourcesPerNodeCache.clear();
--- End diff --
Actually we can reuse these cache for the next scheduling round, when we
bring in central master cache in.
---
