Nachi, this wasn't a rank against missing reviews! I hope nobody gets offended.
My goal is to see if we can do something to improve our turnaround times. The metric we need to improve, I think, are median time to first review and median time to approval/abandon. I'd prefer the median compared to the average as a metric. Salvatore On 28 June 2013 14:54, Nachi Ueno <[email protected]> wrote: > Hi Salvatore > > Thank you for your pointing this. > I'll improve my review counts. > > Best > Nachi > > 2013/6/28 Salvatore Orlando <[email protected]>: > > Not great apparently [1]. > > If this were a soccer league among openstack projects, we would have been > > relegated! > > > > As the things stand right now, I am part of the problem, due to lack of > > reviews [2] > > Even if we could all do with more reviews from the core team (barring the > > ones who are already doing a great job), I reckon that the biggest > problem > > is the wait time for the first review. > > A possible cause, in my opinion, lies in the review days. > > Since each core has a specific area of expertise, patches for that area > > might end up lying in limbo until that core's review day comes. From my > > perspective, I am trying, as much as possible, to allocate a slice of my > > time every day for reviews; another thing I'm doing is to pick patches > from > > the bottom at least one day a week to avoid starvation. > > However, I desperately need to improve my react time to new patchsets, > as I > > often miss them. Gerrit emails are very high volume, so email > notifications > > often get lost. Suggestions are welcome. > > > > Another cause are plugin-specific patches. Some of these reviews, > especially > > for plugins with no core members, sit for a rather long time. It would be > > good if every plugin sub-team lead can nominate an "alternate core dev" > that > > patch authors can ping for reviews. > > > > Salvatore > > > > [1] http://russellbryant.net/openstack-stats/all-openreviews.html > > [2] http://russellbryant.net/openstack-stats/quantum-reviewers-30.txt > > > > -- > > Mailing list: https://launchpad.net/~quantum-core > > Post to : [email protected] > > Unsubscribe : https://launchpad.net/~quantum-core > > More help : https://help.launchpad.net/ListHelp > > >
-- Mailing list: https://launchpad.net/~quantum-core Post to : [email protected] Unsubscribe : https://launchpad.net/~quantum-core More help : https://help.launchpad.net/ListHelp
