[openstack-dev] Please stop reviewing code while asking questions

Robert Collins robertc at robertcollins.net
Sat Apr 25 04:04:25 UTC 2015


On 25 April 2015 at 05:43, Ben Nemec <openstack at nemebean.com> wrote:
> On 04/24/2015 07:11 AM, Russell Bryant wrote:
>> On 04/24/2015 07:21 AM, Amrith Kumar wrote:
>>> We had a hypothesis about why +0 was rarely used (never conclusively
>>> proved). Our hypothesis was that since Stackalytics didn't count +0's
>>> it led to an increased propensity to -1 something. It would be
>>> wonderful if we could try the experiment of giving credit for 0's and
>>> seeing if it changes behavior.
>>
>> I think this makes a lot of sense.  These stats really do drive
>> behavior.  I'd certainly be open to a patch to reviewstats [1] to count
>> +0 comments and I think it would be good for stackalytics to consider
>> the same.
>>
>> [1] http://git.openstack.org/cgit/openstack-infra/reviewstats
>>
>
> I actually looked at doing this a while ago, but unfortunately the json
> data we were getting from Gerrit didn't include no scores.  That may
> have been pre-Gerrit upgrade so it's possible it will have changed, but
> it's worth noting that we may just not have the ability to track this.

I believe we'll need to switch to the REST JSON API, which gertty uses
and gets comments -> some nontrivial refactoring implied, and I
shudder to think about the performance impact.

I've wanted to rewrite reviewstats for a while... this might be the trigger...

-Rob


-- 
Robert Collins <rbtcollins at hp.com>
Distinguished Technologist
HP Converged Cloud



More information about the OpenStack-dev mailing list