Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/1249#issuecomment-201319899
Overall it looks really good. Only one real bug that I could see, but then
some nits and design choice questions, mostly around if what we did for clojure
still makes since in java, all of the design stuff I see as very minor and we
can address it later.
---
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.
---
