Closed
Bug 952199
Opened 11 years ago
Closed 9 years ago
[tracker] topcrasher_ranks_bybug is too slow
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: rhelmer, Unassigned)
References
()
Details
(Keywords: perf)
The performance of the topcrasher_ranks_bybug depends greatly on how many product+version combination this signature appears in, because the current implementation pulls a TCBS report for each product+version+signature combination to get the "current rank" (which is relative to these three dimensions)
Some bugs such as https://crash-stats.allizom.org/topcrasher_ranks_bybug/?bug_number=947213 are fairly quick but https://crash-stats.allizom.org/topcrasher_ranks_bybug/?bug_number=711568 for example is pathologically slow.
Reporter | ||
Comment 1•11 years ago
|
||
From measuring locally it looks like the TCBS call (to get "current rank" as mentioned in comment 0) is the problem here, so the fix is either to make that faster or eliminate the need to make this call (or so many of them).
Comment 2•11 years ago
|
||
It looks to me like we are gathering TCBS for more than the active version set right now, could we reduce to that?
Reporter | ||
Updated•11 years ago
|
Summary: topcrasher_ranks_bybug is too slow → [tracker] topcrasher_ranks_bybug is too slow
Updated•11 years ago
|
Reporter | ||
Comment 3•11 years ago
|
||
Bug 963236 made this *much* faster, for cases where we already have TCBS cached. Even when it still takes a while, loading that URL again should be very quick.
Status: NEW → ASSIGNED
Depends on: 963236
Comment 4•11 years ago
|
||
New results are up at http://www.webpagetest.org/result/140129_YQ_130S/ -- first view = ~ 45 seconds, second view = ~ 2; wow, indeed.
Reporter | ||
Comment 5•9 years ago
|
||
Not sure if this report is going to stick around, or if it's still in use - in any case I haven't been working on it.
Assignee: rhelmer → nobody
Status: ASSIGNED → NEW
Comment 6•9 years ago
|
||
We now have https://mozilla.github.io/bug-signatures-status/
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•