<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 18, 2014 at 5:19 PM, Clint Byrum <span dir="ltr"><<a href="mailto:clint@fewbar.com" target="_blank">clint@fewbar.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Excerpts from Duncan Thomas's message of 2014-06-17 03:56:10 -0700:<br>


<div class="">> A far more effective way to reduce the load of trivial review issues<br>
> on core reviewers is for none-core reviewers to get in there first,<br>
> spot the problems and add a -1 - the trivial issues are then hopefully<br>
> fixed up before a core reviewer even looks at the patch.<br>
><br>
> The fundamental problem with review is that there are more people<br>
> submitting than doing regular reviews. If you want the review queue to<br>
> shrink, do five reviews for every one you submit. A -1 from a<br>
> none-core (followed by a +1 when all the issues are fixed) is far,<br>
> far, far more useful in general than a +1 on a new patch.<br>
><br>
<br>
</div>Perhaps we should incentivize having a good "reviews to patches" ratio<br>
somehow. There are probably quite a few people who are not ever going to<br>
be core reviewers, but who don't mind doing a few reviews per day.<br>
<br></blockquote><div><br></div><div>Perhaps we can add that to <a href="http://stackalytics.com/report/contribution/nova-group/30">http://stackalytics.com/report/contribution/nova-group/30</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


I can think of a few ways, but one way is to make that a real statistic<br>
(brace yourselves for the warnings of "gaming the system") and then give<br>
the top 10 non-core reviews to patches ratios a shout out each release.<br>
<div class=""><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>