[openstack-dev] [metrics] Old reviews (2011) with strange uploaded dates in review.openstack.org

2014-09-12 Thread Daniel Izquierdo

Hi there,

I was checking some datasets [1] by the Activity Board [2] and realized 
that there are some inconsistencies in the old reviews, around July 
2011, at the very beginning.


An example of this [3] shows that the uploaded date is Dec 16, 2012, 
while the review was opened on the 25th of July, 2011.


I know that this is not a big issue nowadays and even more for the day 
to day work of the developers, but for the Activity Board, this was 
producing some negative numbers in the review process, what seemed a bit 
strange.


So, just to point the focus to those dates for those working on metrics 
:). And a question: was there a migration around the 2012-12-16 of the 
review system or some other noticeable event?. On such date there was 
around 1,200 submitted reviews, while around those days, in mean, there 
are some dozens of them.


Cheers,
Daniel.


[1] http://activity.openstack.org/dash/browser/data/db/reviews.mysql.7z
[2] http://activity.openstack.org/dash/browser/
[3] https://review.openstack.org/#/c/44/

--
Daniel Izquierdo Cortazar, PhD
Chief Data Officer
-
Software Analytics for your peace of mind
www.bitergia.com
@bitergia


___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [metrics] Old reviews (2011) with strange uploaded dates in review.openstack.org

2014-09-12 Thread Jeremy Stanley
On 2014-09-12 11:54:19 +0200 (+0200), Daniel Izquierdo wrote:
[...]
 And a question: was there a migration around the 2012-12-16 of the
 review system or some other noticeable event?. On such date there
 was around 1,200 submitted reviews, while around those days, in
 mean, there are some dozens of them.

We discovered that Gerrit configures datestamps in most of its
tables to reset on row updates (a particularly insane design choice
for things like a created_on column). Before we realized this,
intrusive maintenance activities--most notably project renames--were
mass-resetting the creation dates of changes and comments to the
date and time we ran the necessary update queries. Now we
special-case those fields in our update queries to forcibly reset
them to themselves so that they retain their original values, but at
this point there's no easy way to go back and fix the ones we did
before we noticed this unfortunate loss of date/time information.

This maintenance notification looks relevant...

http://lists.openstack.org/pipermail/openstack-dev/2012-December/003934.html

-- 
Jeremy Stanley

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [metrics] Old reviews (2011) with strange uploaded dates in review.openstack.org

2014-09-12 Thread Daniel Izquierdo

On 12/09/14 15:38, Jeremy Stanley wrote:

On 2014-09-12 11:54:19 +0200 (+0200), Daniel Izquierdo wrote:
[...]

And a question: was there a migration around the 2012-12-16 of the
review system or some other noticeable event?. On such date there
was around 1,200 submitted reviews, while around those days, in
mean, there are some dozens of them.

We discovered that Gerrit configures datestamps in most of its
tables to reset on row updates (a particularly insane design choice
for things like a created_on column). Before we realized this,
intrusive maintenance activities--most notably project renames--were
mass-resetting the creation dates of changes and comments to the
date and time we ran the necessary update queries. Now we
special-case those fields in our update queries to forcibly reset
them to themselves so that they retain their original values, but at
this point there's no easy way to go back and fix the ones we did
before we noticed this unfortunate loss of date/time information.
That makes totally sense, thanks a lot for the info!. Then we should try 
to avoid those reviews when calculating time to review and other 
time-based metrics.




This maintenance notification looks relevant...

http://lists.openstack.org/pipermail/openstack-dev/2012-December/003934.html
Oops, thanks for the pointer. It's exactly that date (I didn't check the 
infra mailing list for that exactly date, my fault u_u).


Thanks a lot!

Regards,
Daniel.






--
Daniel Izquierdo Cortazar, PhD
Chief Data Officer
-
Software Analytics for your peace of mind
www.bitergia.com
@bitergia


___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev