Reduce the number of returned search results to from 500 to 250

VERIFIED FIXED in 2.3

Status

VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: mbrandt, Assigned: stephend)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

7 years ago
Would it be possible to tune down the number of results before pagination from 500 to 250? We believe we're seeing inconsistent failures in some of our automated tests due to an out-of-memory error that is occurring in our testing env.

Example query:

https://crash-stats.mozilla.com/report/list?product=Fennec&query_search=signature&query_type=contains&reason_type=contains&date=09%2F19%2F2011%2014%3A41%3A53&range_value=1&range_unit=weeks&hang_type=any&process_type=any&do_query=1&signature=inflate_fast_copy_neon

Comment 1

7 years ago
Pull request from stephenD: https://github.com/stephendonner/socorro/commit/f490e6cc96faf2b262a7e51da0644cc04e9e68f9 

r+

merged as https://github.com/mozilla/socorro/commit/f080a840486b4a6583582ab01919a468040a6b38
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Assignee: nobody → stephen.donner
Target Milestone: --- → 2.3
(Reporter)

Comment 4

7 years ago
Urg ... sorry. I mis-verified this bug. It look like the results set is still returning 500 per page before pagination occurs.

ttps://crash-stats.mozilla.com/report/list?product=Fennec&query_search=signature&query_type=contains&reason_type=contains&date=09%2F19%2F2011%2014%3A41%3A53&range_value=1&range_unit=weeks&hang_type=any&process_type=any&do_query=1&signature=inflate_fast_copy_neon
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
This actually needs to be set in the puppet config (we don't have an easy override system for the PHP app like we do for the python apps).

I just checked this in, will close the bug when we get it pushed:

Committed revision 22037.

The value on stage was 25 for some reason, it's now:

$config['number_report_list'] = 25;

Let's just double-check it looks ok on stage after this bug is resolved and we can do the same on prod.
(Reporter)

Comment 6

7 years ago
Thanks rhelmer ... I had gotten a note from our team in Romania that the counts were off. :-)
Pushed to crash-stats.allizom.org
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
(Reporter)

Comment 8

7 years ago
QA verified with the help of one of brandonsavage's elegant solution. Cheers.
Status: RESOLVED → VERIFIED
Component: Socorro → General
Product: Webtools → Socorro
You need to log in before you can comment on or make changes to this bug.