Last Comment Bug 466059 - Revise search to put limitations on query size
: Revise search to put limitations on query size
Status: RESOLVED FIXED
:
Product: Socorro
Classification: Server Software
Component: General (show other bugs)
: Trunk
: All All
: -- normal (vote)
: 0.6
Assigned To: Austin King [:ozten]
: socorro
Mentors:
Depends on: 466233
Blocks:
  Show dependency treegraph
 
Reported: 2008-11-20 15:44 PST by Michael Morgan [:morgamic]
Modified: 2011-12-28 10:40 PST (History)
0 users
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Adds search constraint (20.97 KB, patch)
2008-11-20 16:42 PST, Austin King [:ozten]
morgamic: review+
Details | Diff | Splinter Review
needed an hours condition... (5.60 KB, patch)
2008-11-20 21:19 PST, Austin King [:ozten]
morgamic: review+
Details | Diff | Splinter Review

Description Michael Morgan [:morgamic] 2008-11-20 15:44:16 PST
Need to limit the breadth of searches so we don't have multiple 3-month queries with no product running on the app db.
Comment 1 Austin King [:ozten] 2008-11-20 16:42:24 PST
Created attachment 349311 [details] [diff] [review]
Adds search constraint
Comment 2 Michael Morgan [:morgamic] 2008-11-20 16:51:53 PST
Comment on attachment 349311 [details] [diff] [review]
Adds search constraint

I tested this last night and this morning.  Looks good, except for one line.

+	<p><strong>Notice:</strong> We are currently limiting search for performance reasons. Please pick 1 or more products. Please choose a date (or date range) within the last 4 weeks. Thanks. If you need to search further back into crash reports history, <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=466059">let us know</a></p>

Delete this line before you check it in.

Thanks, Austin!
Comment 3 Austin King [:ozten] 2008-11-20 16:58:06 PST
Commited r705 (sans extra notice)
Comment 4 Michael Morgan [:morgamic] 2008-11-20 19:07:42 PST
Seems that when I put a 1-hour query in it default to 2-weeks.  Should take a look at this tomorrow.  Sorry for not catching that earlier.
Comment 5 Austin King [:ozten] 2008-11-20 21:19:24 PST
Created attachment 349363 [details] [diff] [review]
needed an hours condition...
Comment 6 Michael Morgan [:morgamic] 2008-11-21 13:19:22 PST
Comment on attachment 349363 [details] [diff] [review]
needed an hours condition...

WFM - thanks, Austin.
Comment 7 Austin King [:ozten] 2008-11-21 13:25:35 PST
Checked in r713
Comment 8 Austin King [:ozten] 2008-11-21 13:28:51 PST
IT Request:
https://bugzilla.mozilla.org/show_bug.cgi?id=466220

Note You need to log in before you can comment on or make changes to this bug.