hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Todd Lipcon (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-8248) Clarify bylaws about review-then-commit policy
Date Mon, 09 Apr 2012 21:05:16 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13250179#comment-13250179

Todd Lipcon commented on HADOOP-8248:

To add a little more: I think the requirement of 3 committer +1s from people who didn't work
on the branch will make it really hard to ever merge branches. Looking, for example, at the
recent HA branch merge, it listed the following people as patch contributors:
bq. Contributed by Todd Lipcon, Aaron T. Myers, Eli Collins, Uma Maheswara Rao G, Bikas Saha,
Suresh Srinivas, Jitendra Nath Pandey, Hari Mankude, Brandon Li, Sanjay Radia, Mingjie Lai,
and Gregory Chanan

Finding 3 active committers who are not on that list and are knowledgeable about NN internals
would have been very difficult. In fact of the committers who did +1 the merge, you're the
only one who isn't in the above list :)
> Clarify bylaws about review-then-commit policy
> ----------------------------------------------
>                 Key: HADOOP-8248
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8248
>             Project: Hadoop Common
>          Issue Type: Task
>            Reporter: Todd Lipcon
>         Attachments: c8248_20120409.patch, proposed-bylaw-change.txt
> As discussed on the mailing list (thread "Requirements for patch review" 4/4/2012) we
should clarify the bylaws with respect to the review-then-commit policy. This JIRA is to agree
on the proposed change.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


View raw message