Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2657#discussion_r186169276
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/ConstraintSolverStrategy.java
---
@@ -173,6 +175,7 @@ private static boolean
checkSpreadSchedulingValid(Cluster cluster, TopologyDetai
*/
private static boolean checkResourcesCorrect(Cluster cluster,
TopologyDetails topo) {
LOG.info("Checking Resources...");
+ assert(cluster.getAssignmentById(topo.getId())!=null);
--- End diff --
nit: same here
---
