[openstack-dev] RFC: reverse the default Gerrit sort order

Monty Taylor mordred at inaugust.com
Mon Nov 11 04:36:35 UTC 2013

On 11/10/2013 11:06 PM, Clark Boylan wrote:
> On Sun, Nov 10, 2013 at 6:18 PM, Robert Collins
> <robertc at robertcollins.net> wrote:
>> On 11 November 2013 10:06, Monty Taylor <mordred at inaugust.com> wrote:
>>> Not possible now. With gerrit 2.8 (hopefully something we can get
>>> deployed soonish) we introduce the Custom Dashboards feature. The
>>> "Important Changes" page will be implemented using that, which means we
>>> should be able to play with things like you suggest above.
>>> That said:
>>> https://review.openstack.org/#/mine/important/
>>> Shows me old changes at the top of the reviewable section. Do you use
>>> that view at all?
>> I don't because:
>>  - they seem to be sorted by patch #, not by last-comment or last-push
>>  - the reviewable by me section includes all nova open reviews (as I watch it)
>>  - the reviewed by me section is also a 100 or so long
>> -> useless.
>> I would like to review things I previously reviewed first, but
>> re-reading > 100 reviews to manually detect which ones had replies ==
>> painful.
>> -Rob
> They are sorted on the reverse of the default sort order (which is
> change number IIRC but would need to reread code to be sure). The
> reliance on watched projects is probably one of the biggest problems
> with this dashboard view, but that is what watched projects exist for.
> I have heard suggestions that different levels of watching may be
> helpful, but I doubt upstream would take patches that did that (worth
> discussing with them though).
> Recently I have been using stars to aggressively mark the items that I
> think can move through the queue, I review them then keep them starred
> until they are merged. If they move too slowly they get unstarred.
> This is typically a much smaller subset of the larger queue and makes
> dealing with context switching much easier. Doesn't quite solve your
> second point about detecting which items have replies, but tackles a
> similar problem, tracking items that are under development and moving.

I think the use cases and desires are divergent enough that 2.8's user
dashboards feature will be the best bet. Then, if we get that going,
further "I can't filter/sort by X" things are distinct and specific
upstream bugs.

In the meantime, I've been making complex search queries in the search
box and then just saving the resulting URL as 'saved search' link.


If you haven't seen it. Best I can tell, this will still not solve your
desire, for two reasons:

- "I have previously reviewed the patch" is not a searchable term. (I
know, because I'd like a view of "things I have not reviewed yet")

- There doesn't seem to be any user-consumable way to specify sort.

More information about the OpenStack-dev mailing list