search

berkus / rietveld

Code review tool from Guido for Django
code.google.com/p/rietveld
Apache License 2.0
0 stars 0 forks source link

/user/me/stats is broken #482

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
The cron job is not running successfully anymore.

Code:
https://code.google.com/p/rietveld/source/browse/codereview/views.py#3891
Entities:
https://code.google.com/p/rietveld/source/browse/codereview/models.py#1283

Side note: for heavy reviewers, the entities become significantly large 
(especially for the ones doing hundreds of reviews per month). This results in 
the DB being overloaded when trying to view stats for this users. A new model 
should be generated that contants the cache of the overall stats, e.g. all the 
properties of AccountStatsBase saved as an entities but without .issues, 
.latencies, .lgtms and .review_types. This needs to be a new type, which would 
be a cache of the actual raw data.

Original issue reported on code.google.com by maruel@chromium.org on 14 Mar 2014 at 4:12

GoogleCodeExporter commented 9 years ago 
I have the stats cron job actually executing on codereview.chromium.org and is 
should start running on codereview.appspot.com after the next release.

However, the stats shown on the /stats page are all 0.

Original comment by jrobbins@google.com on 11 Jul 2014 at 6:53