Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/401#issuecomment-72102590
For the most part it looks good, I would like to see us use the cached
assignment instead of going to zookeeper again to get it. I would also like to
understand better how we avoid issues with downloading the same thing in two
different threads. I'm not sure I understand the exact cause of what puts us
in this situation to begin with, so I don't feel comfortable that we will not
try to download the same thing twice and cause some issues when we do.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
