Closed Bug 1261010 Opened 8 years ago Closed 3 years ago

[Topcrashers] Pass list of signatures to second query

Categories

(Socorro :: Webapp, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: adrian, Unassigned, Mentored)

Details

(Whiteboard: [good)

In the new, Super Search based topcrashers app, we make two queries. The first one gets the top signatures for the time period. The second one gets the top signature for the previous time period, in order to compare ranks and numbers and show progression of signatures ranks. 

At the moment, that second query multiplies the facet size by 2 to increase the chances of seeing all the signatures of the first query, but that might still fail if a signature was really low in the previous time period. We can make that better by simply passing the list of signatures from the first query to the second query. 

The file is ./webapp-django/crashstats/topcrashers/views.py - https://github.com/mozilla/socorro/blob/master/webapp-django/crashstats/topcrashers/views.py

The function is ``get_topcrashers_results``. Instead of ``params['_facets_size'] *= 2``, add a ``params['signature']`` and pass it a list of all signatures, each one prefixed with the "is exactly" operator, symbolized by the equal sign "=". For example: 

> print(params['signature'])
> ['=moz::foo', '=my_func()<>']

I think this makes sense, but I'm going to mark it WONTFIX--it's not something I'm going to do.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.