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.
---
