Merge lp:~deryck/launchpad/ubuntu-bugs-front-timeout-590992 into lp:launchpad
Proposed by
Deryck Hodge
| Status: | Merged |
|---|---|
| Approved by: | Deryck Hodge |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 10968 |
| Proposed branch: | lp:~deryck/launchpad/ubuntu-bugs-front-timeout-590992 |
| Merge into: | lp:launchpad |
| Diff against target: |
15 lines (+0/-5) 1 file modified
lib/lp/bugs/browser/bugtarget.py (+0/-5) |
| To merge this branch: | bzr merge lp:~deryck/launchpad/ubuntu-bugs-front-timeout-590992 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Björn Tillenius (community) | code/rc | Approve | |
|
Review via email:
|
|||
Commit message
Remove an unused bug count variable that is causing timeouts on Ubuntu bugs home.
Description of the change
bugs.lp.net/ubuntu is timing out. See bug 590992.
This branch removes an unused bug_count_items from browser code.
Getting this bug count for all Ubuntu bugs was causing the timeout. I
I have tested on staging and confirmed that the timeout is fixed with
dropping these lines. Spot checks of tests see no fallout from
removing the code.
The branch is going off to ec2 as I type this MP.
I'n also interested in getting RC approval and getting this change
cowboyed onto lpnet and edge to get the bugs home working again until
the branch can work its way through ec2 and landing.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
< BjornT> deryck: have you confirmed that that view isn't used on other
pages?
< deryck> BjornT, the view is for every top-level bugs page, not just
distro. a grep for bug_count_items turns up nothing. doc and
browser tests pass. I'll look at zcml to make sure nothing
else uses it, though....