The distroseries +packaging times out
Bug #492012 reported by
Curtis Hovey
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Unassigned |
Bug Description
Oops OOPS-1433EC1075 shows that there is not wrong with the SQL, there are too many packaging links to render (1500). The list should be batch 200-500 long. This page also show the unlinked translatables, which is short and does not need to be shown on subsequent batches.
Related branches
lp:~sinzui/launchpad/dsp-packaging-bug-492012
Merged
into
lp:launchpad
- Aaron Bentley (community): Approve
-
Diff: 268 lines (+144/-30)5 files modifiedlib/lp/registry/browser/distroseries.py (+30/-11)
lib/lp/registry/browser/tests/packaging-views.txt (+55/-0)
lib/lp/registry/stories/distroseries/xx-show-distroseries-packaging.txt (+47/-13)
lib/lp/registry/stories/packaging/xx-sourcepackage-packaging.txt (+1/-1)
lib/lp/registry/templates/distroseries-packaging.pt (+11/-5)
lp:~sinzui/launchpad/false-packages
Merged
into
lp:launchpad
- Edwin Grubbs (community): Approve (code)
-
Diff: 111 lines (+29/-27)4 files modifiedlib/lp/registry/browser/distroseries.py (+0/-5)
lib/lp/registry/browser/tests/packaging-views.txt (+2/-8)
lib/lp/registry/model/distroseries.py (+25/-13)
lib/lp/registry/stories/distroseries/xx-show-distroseries-packaging.txt (+2/-1)
Changed in launchpad-registry: | |
status: | Triaged → In Progress |
assignee: | nobody → Curtis Hovey (sinzui) |
Changed in launchpad-registry: | |
status: | In Progress → Fix Committed |
Changed in launchpad: | |
assignee: | Curtis Hovey (sinzui) → nobody |
To post a comment you must log in.
Adding batch navigator and filtering the unversioned links seems to have made the timeouts worse. I think the fix needs to do down to the model.