[PROPOSAL] Tag Pull Request with feature tag

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[PROPOSAL] Tag Pull Request with feature tag

Jacky Li
Hi All,

As there are more and more PRs coming to the Apache Carbon repository, the review and commit cycle of one PR takes longer time than before. One reason of this is because sometimes it is hard for committer to find the PR to review for certain feature. And in many cases PRs for same feature depends on one and another, it needs to be reviewed in an correct order, which makes this situation even worse.

To make the review more efficient and encourage developer to contribute more, I propose to include one optional tag in PR to reflect which feature the PR belongs to, like:

[CARBONDATA-JiraTicketNumer][FeatureName] title of the PR.  

For example, to raise a PR for aggregate table feature, one would raise a PR with title:
[CARBONDATA-1234][AggTable] Support create aggregate table

By doing this, developer will have better idea of which feature does this PR contribute to.

As an example, I propose following feature name for 1.3 release.

1. AggTable
2. Streaming
3. DataMap

Please note that this feature name tag in the PR is not mandatory, and it is for committer's reference only.
New feature name can be discussed via this maillist at anytime.


Regards,
Jacky

Reply | Threaded
Open this post in threaded view
|

Re: [PROPOSAL] Tag Pull Request with feature tag

Liang Chen
Administrator
+1, agree with this proposal.

Regards
Liang



--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/
Reply | Threaded
Open this post in threaded view
|

Re: [PROPOSAL] Tag Pull Request with feature tag

sraghunandan
+1
On Sat, 28 Oct 2017 at 8:05 PM, Liang Chen <[hidden email]> wrote:

> +1, agree with this proposal.
>
> Regards
> Liang
>
>
>
> --
> Sent from:
> http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/
>
Reply | Threaded
Open this post in threaded view
|

Re: [PROPOSAL] Tag Pull Request with feature tag

xm_zzc
In reply to this post by Jacky Li
+1, we can add a new feature tag called "Spark2.2", OK?



--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/
Reply | Threaded
Open this post in threaded view
|

回复: [PROPOSAL] Tag Pull Request with feature tag

cenyuhai11
+1






Best regards!
Yuhai Cen


在2017年10月30日 10:18,xm_zzc<[hidden email]> 写道:
+1, we can add a new feature tag called "Spark2.2", OK?



--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/