Merge security branch into master

2014-11-12 Thread Bobby Evans
I think the pull request to add authentication and authorization to storm is 
now ready to be merged into master.

https://github.com/apache/storm/pull/121#issuecomment-62754593
It is a little different from most pull requests, because of its size and all.  
As I said in the comment  but I would like to see what the community thinks of 
it.  Please comment on the pull request itself so that it appears in the JIRA 
too.

 Thanks,

BobbyEvans


Re: Merge security branch into master

2014-11-12 Thread P. Taylor Goetz
I'm +1 for merging the security branch to master, we just need a little 
coordination.

Tomorrow (EST) I'll create a 0.9.3-branch from the current master. After that 
any outstanding pull requests targeting master should be merged to that branch 
(obviously +1, etc. rules apply).

That way we can continue iterating on the 0.9.3 release as necessary to make 
sure it is solid, and still move forward with the 0.10 (security) work.

How does that sound?

-Taylor


 On Nov 12, 2014, at 12:17 PM, Bobby Evans ev...@yahoo-inc.com.INVALID wrote:
 
 I think the pull request to add authentication and authorization to storm is 
 now ready to be merged into master.
 
 https://github.com/apache/storm/pull/121#issuecomment-62754593
 It is a little different from most pull requests, because of its size and 
 all.  As I said in the comment  but I would like to see what the community 
 thinks of it.  Please comment on the pull request itself so that it appears 
 in the JIRA too.
 
 Thanks,
 
 BobbyEvans