Merge lp:~cjwatson/launchpad/front-page-bulk into lp:launchpad
Proposed by
Colin Watson
Status: | Merged |
---|---|
Merged at revision: | 18504 |
Proposed branch: | lp:~cjwatson/launchpad/front-page-bulk |
Merge into: | lp:launchpad |
Diff against target: |
192 lines (+57/-10) 4 files modified
lib/lp/app/browser/tests/test_launchpadroot.py (+22/-1) lib/lp/registry/model/pillar.py (+27/-5) lib/lp/testing/__init__.py (+4/-1) lib/lp/testing/_login.py (+4/-3) |
To merge this branch: | bzr merge lp:~cjwatson/launchpad/front-page-bulk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
William Grant | code | Approve | |
Review via email: mp+333510@code.launchpad.net |
Commit message
Preload pillars and icons in PillarNameSet.
Description of the change
This gets rid of well over half the queries on the front page (about 50 out of 80).
Since this model property has exactly one user, I just made the eager loading mandatory rather than doing anything more clever.
To post a comment you must log in.