Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2203
Sorry about not responding earlier. Thanks for addressing my comments.
I skimmed the comments, and found a question that might have been missed
https://github.com/apache/storm/pull/2203/files#r155645156. It's not that I
think we necessarily have to make a change there, but I'd like to understand
whether we're trying to guard against misconfiguration or something else.
Other than that nit this looks great. +1.
Since it's come up a few times, will documentation go here or in a later PR?
---
