Vel, thanks for putting this forward. I have updated the Wiki page 
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=55151244 with 
your comments. Let’s keep this update as we go.

I personally feel there are too many steps, which might be overlooked by some. 
I couldn’t find steps which can remove, except make a few as optional (or 
default).

I also suggest, we should eventually go towards git pull request. It is used by 
most of the recent Apache projects and it is more convenient to review and 
commit.

Thanks

Bosco


On 12/21/16, 4:27 AM, "Velmurugan Periasamy" <[email protected]> wrote:

    Rangers:
    
    I would like to share few pointers for submitting a patch via JIRA and
    Apache review board. These steps will help review process to be more
    effective. Please add if you have any other suggestions.
    
    ‹ Add Apache JIRA # in review request title.
    ‹ Specify branch information (master/ranger-0.6 etc) and Apache JIRA # in
    the review request fields.
    ‹ Provide a link to the review request in Apache JIRA.
    ‹ Attach the patch to Apache JIRA. If there are updated versions (based on
    review suggestions), make sure to include the latest patch.
    ‹ Mark the fix version appropriately (if you know).
    ‹ Update the JIRA with commit link once the patch is reviewed and committed.
    ‹ Mark the review request as closed once the patch is committed. This will
    help to keep review dashboard view clean, so that open patches get the right
    attention. 
    
    Thank you,
    Velmurugan Periasamy
    
    
    
    
    


Reply via email to