hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From yongqiang he <>
Subject Re: patch review process
Date Thu, 20 Jan 2011 02:48:45 GMT
+1 for option 2.

In general, we as a community should be nice to all contributors, and
should avoid doing things that make contributors not comfortable, even
that requires some work from committers. Sometimes it is especially
true for new contributors, like we need to be more patience for new
people. It seems a free style and contribution focused environment
would be better to encourage people to do more contributions of
different kinds.

On Wed, Jan 19, 2011 at 6:37 PM, Namit Jain <> wrote:
> It would be good to have a policy for submitting a new patch for review.
> If the patch is small, usually it is pretty easy to review.But, if it large,
> a GUI like reviewboard ( makes it easy.
> So, going forward, I would like to propose either of the following.
> 1. All patches must go through reviewboard
> 2. If a contributor/reviewer creates a reviewboard request,
>     all subsequent review requests should go through the reviewboard.
> I would personally vote for 2., since for small patches, we don’t really need a
> reviewboard.
> But, please vote, and based on that, we can come up with a policy.
> Let us know, if you think of some other option.
> Thanks,
> -namit

View raw message