Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2182
@kevinconaway
I don't mind the change is not covered by test since the change is really
small, but adding a new test is better for sure.
The change looks good, but I didn't have time to check the test code. I'll
revisit this soon.
Btw, I'd like to see the PR against master branch too. Please note that
checkstyle is applied to the master branch.
---
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.
---
