Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2723
@revans2
Hi, bobby, i have already updated the test data for this patch.
As for NeedsFullTopologiesScheduler, i agree to remove it. We can tweak the
IsolationScheduler or cache topologies data, either is ok.
But i am inclined to choose the first one, cause we do not have a powerful
reliable cache for master now, everything we cached will need to have a recover
strategy, too many cache will make this things a little mess.
For the first chose, i think more work will be done for refactoring the
code, I will fire a another JIRA to solve this problem.
For this patch, i think it is enough for us.
---
