Last Comment Bug 853483 - Triage report times out on Firefox (Any) query
: Triage report times out on Firefox (Any) query
Classification: Other
Component: General (show other bugs)
: Production
: x86 Mac OS X
-- normal (vote)
: ---
Assigned To: Byron Jones ‹:glob›
Depends on:
  Show dependency treegraph
Reported: 2013-03-21 09:11 PDT by Liz Henry (:lizzard) (needinfo? me)
Modified: 2013-04-23 21:53 PDT (History)
4 users (show)
See Also:
Due Date:
QA Whiteboard:
Iteration: ---
Points: ---

patch v1 (88.45 KB, patch)
2013-03-25 08:28 PDT, Byron Jones ‹:glob›
dkl: review+
Details | Diff | Splinter Review

Description User image Liz Henry (:lizzard) (needinfo? me) 2013-03-21 09:11:00 PDT
At , when I request the Firefox report for _Any_ , where the last commenter is the reporter, the query fails with the message, "Service Unavailable. The service is temporarily unavailable. Please try again later."  

Narrower queries still work.
Comment 1 User image Byron Jones ‹:glob› 2013-03-25 08:28:15 PDT
Created attachment 728991 [details] [diff] [review]
patch v1

the problem here is the amount of bugs it has to process is too large, causing a timeout in the load balancer.  to address this i've set an upper-limit on the number of bugs.

i also took the opportunity to do some long-overdue refactoring of the custom reports by breaking out each set of reports into their own files.
Comment 2 User image David Lawrence [:dkl] 2013-04-23 14:54:12 PDT
Comment on attachment 728991 [details] [diff] [review]
patch v1

Review of attachment 728991 [details] [diff] [review]:

Just minor stuff. Otherwise the reports worked fine for me and the new error message worked as well. r=dkl

::: extensions/BMO/template/en/default/pages/email_queue.html.tmpl
@@ +47,4 @@
>          [% END %]
>        </td>
>        <td>
> +        [% job.subject ? job.subject : '-' FILTER html %]


@@ +52,5 @@
>      </tr>
> +    [% IF job.subject %]
> +      <tr class="report item [% loop.count % 2 == 1 ? "report_row_odd" : "report_row_even" %]">
> +        <td colspan="3">&nbsp;</td>
> +        <td colspan="3">[% job.subject FILTER html %]</td>

I feel this would look better if you used the entire row for the subject. For example:

<td colspan="6">Subject: [% job.subject FILTER html %]</td>
Comment 3 User image Byron Jones ‹:glob› 2013-04-23 21:53:27 PDT
Committing to: bzr+ssh://
modified extensions/BMO/
added extensions/BMO/lib/Reports
missing extensions/BMO/lib/
deleted extensions/BMO/lib/
added extensions/BMO/lib/
added extensions/BMO/lib/Reports/
added extensions/BMO/lib/Reports/
added extensions/BMO/lib/Reports/
added extensions/BMO/lib/Reports/
added extensions/BMO/lib/Reports/
modified extensions/BMO/template/en/default/hook/global/user-error-errors.html.tmpl
modified extensions/BMO/template/en/default/pages/email_queue.html.tmpl
Committed revision 8756.

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