<div dir="ltr">On Wed, Nov 6, 2013 at 3:36 PM, Robert Collins <span dir="ltr"><<a href="mailto:robertc@robertcollins.net" target="_blank">robertc@robertcollins.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<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've been thinking about review queues recently (since here at the<br>


summit everyone is talking about reviews! :)).<br>
<br>
One thing that struck me today was that Gerrit makes it easier to<br>
review the newest changes first, rather than the changes that have<br>
been in the queue longest, or the changes that started going through<br>
the review process first.<br>
<br>
So... is it possible to change the default sort order for Gerrit? How<br>
hard is it to do - could we do an experiment on that and see if it<br>
nudges the dial for reviewstats (just make the change, not ask anyone<br>
to change their behaviour)?<br>
<br>
As for what sort order to choose, I'd be happy just getting data on a<br>
different default sort order - it seems like the easiest thing would<br>
be to reverse the current order, vs doing something more<br>
sophisticated.<br>
<br>
If folk are about to jump up and say 'hey, reviewday' or 'hey<br>
next-review' : I specifically want to see what the effect on changing<br>
the Gerrit web UI is, because my sense is that that is the default<br>
place folk do reviews, and I want to change the default-experience<br>
folk have, not the optional experience folk can opt into.<br>
<br>
-Rob<br>
<span class=""><font color="#888888"><br>
--<br>
Robert Collins <<a href="mailto:rbtcollins@hp.com">rbtcollins@hp.com</a>><br>
Distinguished Technologist<br>
HP Converged Cloud<br></font></span></blockquote><div> </div><div><br></div><div>I tend to use this review priority page [1], which appears to give a higher score to things by age (according to the cinder listing the oldest one has no bp or bug and is on the top), bp priority, and bug priority.</div>

<div><br></div><div>Now this doesn't stop people from grabbing the first thing on gerrit reviews, but perhaps we can encourage better practices by where people usually start [2] and maybe a link to good review workflows [3] on gerrit itself?</div>

<div><br></div><div>[1] - <a href="http://status.openstack.org/reviews">http://status.openstack.org/reviews</a></div><div>[2] - <a href="https://wiki.openstack.org/wiki/How_To_Contribute">https://wiki.openstack.org/wiki/How_To_Contribute</a></div>

<div>[3] - <a href="https://wiki.openstack.org/wiki/ReviewWorkflowTips">https://wiki.openstack.org/wiki/ReviewWorkflowTips</a></div><div><br></div><div>--</div><div>Mike Perez</div></div></div></div>