~person/+participation timeouts
Bug #607879 reported by
Robert Collins
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
j.c.sackett | ||
Bug Description
https:/
https:/
https:/
a cursory glance suggests death-by-sql - 826 separate sql queries - and on edge.
Filing as high etc as per the 'zerooopspolicy' - please let me know if thats not appropriate.
Related branches
lp://qastaging/~jcsackett/launchpad/plus-participation-timeouts
- Curtis Hovey (community): Approve (code)
-
Diff: 212 lines (+73/-43)5 files modifiedlib/lp/registry/browser/person.py (+34/-14)
lib/lp/registry/browser/tests/test_person_view.py (+1/-1)
lib/lp/registry/model/person.py (+18/-22)
lib/lp/registry/templates/person-participation.pt (+2/-6)
lib/lp/registry/tests/test_person.py (+18/-0)
lp://qastaging/~jcsackett/launchpad/plus-participation-additional-fixes
- Jelmer Vernooij (community): Approve (code)
- Edwin Grubbs (community): Approve (code)
- Māris Fogels (community): Needs Information
-
Diff: 656 lines (+238/-87)13 files modifiedlib/canonical/launchpad/doc/sample-data-assertions.txt (+2/-2)
lib/lp/bugs/doc/initial-bug-contacts.txt (+1/-1)
lib/lp/bugs/templates/person-portlet-team-assignedbugs.pt (+1/-1)
lib/lp/registry/browser/person.py (+34/-21)
lib/lp/registry/browser/tests/test_person_view.py (+2/-2)
lib/lp/registry/doc/person-account.txt (+2/-2)
lib/lp/registry/doc/private-team-visibility.txt (+1/-1)
lib/lp/registry/doc/teammembership.txt (+5/-5)
lib/lp/registry/doc/vocabularies.txt (+1/-1)
lib/lp/registry/interfaces/person.py (+7/-7)
lib/lp/registry/model/person.py (+77/-15)
lib/lp/registry/tests/test_person.py (+95/-7)
lib/lp/registry/vocabularies.py (+10/-22)
| Changed in launchpad-registry: | |
| milestone: | none → 10.08 |
| Changed in launchpad-registry: | |
| assignee: | nobody → Robert Collins (lifeless) |
| assignee: | Robert Collins (lifeless) → nobody |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #1 |
Revision history for this message
| Robert Collins (lifeless) wrote Re: [Bug 607879] Re: https://bugs.edge.launchpad.net/~person/+participation timeouts | #2 |
| tags: | removed: oops |
| Changed in launchpad-registry: | |
| status: | Triaged → In Progress |
| assignee: | nobody → j.c.sackett (jcsackett) |
| summary: |
- https://bugs.edge.launchpad.net/~person/+participation timeouts + ~person/+participation timeouts |
Revision history for this message
| Robert Collins (lifeless) wrote | #3 |
| Changed in launchpad-registry: | |
| milestone: | 10.08 → 10.09 |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #4 |
| tags: | added: qa-needstesting |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #5 |
| Changed in launchpad-registry: | |
| status: | Fix Committed → In Progress |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
| status: | Fix Committed → In Progress |
| tags: |
added: qa-untestable removed: qa-ok |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #6 |
| tags: |
added: qa-needstesting removed: qa-untestable |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad-registry: | |
| status: | Fix Committed → In Progress |
| tags: |
added: qa-untestable removed: qa-needstesting |
Revision history for this message
| Ursula Junque (ursinha) wrote Re: [Bug 607879] Re: ~person/+participation timeouts | #8 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Incremental fix | #9 |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Robert Collins (lifeless) wrote | #10 |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The problem started when I added mailing lists. We could remove them or batch the page to make a fast fix. The indirect team path has always been costly and requires a lot of queries. The path could be dropped or may loaded via ajax.