<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 8, 2016 at 4:54 PM, Dolph Mathews <span dir="ltr"><<a href="mailto:dolph.mathews@gmail.com" target="_blank">dolph.mathews@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br><br>On Friday, April 8, 2016, John Dickinson <<a href="mailto:me@not.mn" target="_blank">me@not.mn</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 8 Apr 2016, at 13:35, Jeremy Stanley wrote:<br>
<br>
> On 2016-04-08 19:42:18 +0200 (+0200), Dmitry Tantsur wrote:<br>
>> There are many ways to game a simple +1 counter, such as +1'ing changes<br>
>> that already have at least 1x +2, or which already approved, or which need<br>
>> rechecking...<br>
> [...]<br>
><br>
> The behavior which baffles me, and also seems to be on the rise<br>
> lately, is random +1 votes on changes whose commit messages and/or<br>
> status clearly indicate they should not merged and do not need to be<br>
> reviewed. I suppose that's another an easy way to avoid the dreaded<br>
> "disagreements" counter?<br>
> --<br>
> Jeremy Stanley<br>
<br>
<br>
I have been told that some OpenStack on boarding teaches new members of the community to do reviews. And they say, effectively, "muddle through as you can. You won't understand it all at first, but do your best. When you're done, add a +1 and move to the next one"</blockquote><div><br></div></span><div>I advocate for basically this, but instead of a +1, leave a +0 and ask questions. The new reviewer will inevitably learn something and the author will benefit by explaining their change (teaching is the best way to learn).<span></span></div><div class="HOEnZb"><div class="h5"><div> </div></div></div></blockquote><div><br></div><div>This is exactly what I tell people to do as well! Definitely a good direction to encourage folks to go. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I've been working to correct this when I've seen it, but +1 reviews with no comments might not be people trying to game. It might simply be people trying to get involved that don't know any better yet.<br>
<br>
--John<br>
<br>
<br>
<br>
</blockquote>
</div></div><br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div>