Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/121#issuecomment-62754593 I think we are at the point where we should merge the security branch into master. This is a little bit different from most patches/pull requests we have done, partly because of the size of it, and partly because of the large number of contributions from different individuals. All of the code changes individually have been reviewed by storm committers before being pulled into this branch, but I would really like to see at least 2 committers besides myself give a +1 to pulling this in. I think it would be great to have more people look at it, but unless they are blocking issues I would prefer that we file follow up JIRA for them, and get this in so I don't have to keep maintaining it. I am +1.
--- 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. ---