Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Fri, 27 Jan 2017 05:48:25 +0000 (UTC)
From: "stack (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.13038350.1485486659000.13517.1485496105160@Atlassian.JIRA>
In-Reply-To: <JIRA.13038350.1485486659000@Atlassian.JIRA>
References: <JIRA.13038350.1485486659000@Atlassian.JIRA> <JIRA.13038350.1485486659490@jira-lw-us.apache.org>
Subject: [jira] [Commented] (HBASE-17552) Update developer section in hbase
 book
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Fri, 27 Jan 2017 05:50:11 -0000


    [ https://issues.apache.org/jira/browse/HBASE-17552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15841811#comment-15841811 ] 

stack commented on HBASE-17552:
-------------------------------

If doing another cut, 1.0 has been EOL'd. 

Soften this:

Refactor-only patches are exempted.

to .. 'It is hard to get review of refactor-only patches.'

bq. Make sure unit tests pass locally before submitting the patch, which is faster than waiting for presubmit results. This will save your own time and effort.

I'd change this. Our tests take an obnoxiously long time to run. New contributors may be surprised. Suggest that they use jenkins to run the tests because the full suite takes so long (don't worry if the run fails or breaks).

bq. Be sure to get the issue ID right, as this causes Jira to link to the change in Git (use the  issue's "All" tab to see these).

In  here, point at your nice submit-patch scripts (smile).

--signoff is for when the patch is not your own?

Excellent [~appy]

> Update developer section in hbase book
> --------------------------------------
>
>                 Key: HBASE-17552
>                 URL: https://issues.apache.org/jira/browse/HBASE-17552
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Appy
>            Assignee: Appy
>         Attachments: HBASE-17552.master.001.patch
>
>
> Updates
> - Changes 'Create Patch' in major way. Promoting use of submit-patch.py script.
> - Changes instructions for committing patch to make contributor of a patch also  its author so proper credit is given to contributors in terms of github history.
> - Rewording in 'code formatting guidelines'


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)