Github user d2r commented on the pull request:
https://github.com/apache/incubator-storm/pull/36#issuecomment-52406341
* build and ran all tests with this change merged to master, passed
* set up a cluster with two supervisors with 6 slots each, and launched
word_count twice. Each word_count topology uses 3 workers, so I verified that
after the second one is launched, the number of slots used for both of the
supervisors is 3. Without this change, the same test results in 4 slots used
by one supervisor, and 2 by the other.
+1, and thanks for your patience, @xiaokang
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
