[openstack-dev] [nova] A modest proposal to reduce reviewer load

Joe Gordon joe.gordon0 at gmail.com
Thu Jun 19 00:31:36 UTC 2014


On Wed, Jun 18, 2014 at 5:19 PM, Clint Byrum <clint at fewbar.com> wrote:

> Excerpts from Duncan Thomas's message of 2014-06-17 03:56:10 -0700:
> > A far more effective way to reduce the load of trivial review issues
> > on core reviewers is for none-core reviewers to get in there first,
> > spot the problems and add a -1 - the trivial issues are then hopefully
> > fixed up before a core reviewer even looks at the patch.
> >
> > The fundamental problem with review is that there are more people
> > submitting than doing regular reviews. If you want the review queue to
> > shrink, do five reviews for every one you submit. A -1 from a
> > none-core (followed by a +1 when all the issues are fixed) is far,
> > far, far more useful in general than a +1 on a new patch.
> >
>
> Perhaps we should incentivize having a good "reviews to patches" ratio
> somehow. There are probably quite a few people who are not ever going to
> be core reviewers, but who don't mind doing a few reviews per day.
>
>
Perhaps we can add that to
http://stackalytics.com/report/contribution/nova-group/30


> I can think of a few ways, but one way is to make that a real statistic
> (brace yourselves for the warnings of "gaming the system") and then give
> the top 10 non-core reviews to patches ratios a shout out each release.
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140618/4a417fce/attachment.html>


More information about the OpenStack-dev mailing list