Github user Sergeant007 commented on the pull request:
https://github.com/apache/storm/pull/304#issuecomment-61764038
Thanks for the review, @clockfly
I have added necessary comment and removed the tests. Sorry, I wasn't able
to simplify them - if they were in simple synchronous mode, the tests would
hang infinitely if smth. is wrong instead of failures. So I've implemented them
in quite complicated (since I'm new in Clojure), but robust way. Another issue
was in complexity of reproducing of the bug: 1. you should be already connected
and 2. it is reproduced only if you send several messages at once. Anyway, I
have removed the tests as you asked me. Please, review.
---
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.
---
