Fw: Requirements for patch review

2012-04-05 Thread Tsz Wo (Nicholas), Sze
Resent. - Forwarded Message - From: Tsz Wo Sze To: "common-dev@hadoop.apache.org" Cc: Sent: Wednesday, April 4, 2012 8:24 PM Subject: Re: Requirements for patch review >> The wording here is ambiguous, though, whether the committer who >> provides the minimum

Re: Requirements for patch review

2012-04-04 Thread Todd Lipcon
I filed HADOOP-8248 with a diff to the bylaws. Thanks -Todd On Wed, Apr 4, 2012 at 3:26 PM, Eli Collins wrote: > On Wed, Apr 4, 2012 at 2:12 PM, Todd Lipcon wrote: >> Hi folks, >> >> Some discussion between Nicholas, Aaron, and me started in the >> comments of HDFS-3168 which I think is better

Re: Requirements for patch review

2012-04-04 Thread Eli Collins
On Wed, Apr 4, 2012 at 2:12 PM, Todd Lipcon wrote: > Hi folks, > > Some discussion between Nicholas, Aaron, and me started in the > comments of HDFS-3168 which I think is better exposed on the mailing > list instead of trailing an already-committed JIRA. > > The question at hand is what the policy

Re: Requirements for patch review

2012-04-04 Thread Robert Evans
I personally like the clarification and it is in line with how I understood the original bylaw when I read it. I don't really want this to turn into a legal document but as this is getting more explicit with clarification it would be nice to put in a small exception for release managers when th

Requirements for patch review

2012-04-04 Thread Todd Lipcon
Hi folks, Some discussion between Nicholas, Aaron, and me started in the comments of HDFS-3168 which I think is better exposed on the mailing list instead of trailing an already-committed JIRA. The question at hand is what the policy is with regarding our review-then-commit policies. The bylaws s