Merge lp:~wgrant/launchpad/unuse-spr-copyright into lp:launchpad
Proposed by
William Grant
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Robert Collins | ||||||||
Approved revision: | no longer in the source branch. | ||||||||
Merged at revision: | 12576 | ||||||||
Proposed branch: | lp:~wgrant/launchpad/unuse-spr-copyright | ||||||||
Merge into: | lp:launchpad | ||||||||
Diff against target: |
55 lines (+24/-2) 2 files modified
database/schema/security.cfg (+1/-1) lib/lp/soyuz/model/sourcepackagerelease.py (+23/-1) |
||||||||
To merge this branch: | bzr merge lp:~wgrant/launchpad/unuse-spr-copyright | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Robert Collins (community) | Approve | ||
Review via email: mp+52808@code.launchpad.net |
Commit message
[r=lifeless][bug=732424] Stop loading SPR.copyright every time we load an SPR; it can get big.
Description of the change
SPR.copyright can get fairly large on production (up to 2MB), and nothing uses it. Grabbing it from the DB is significant overhead, so this branch prevents that. I turned it from a Storm column into a property that issues a separate query to get and set. archiveuploader and gina seem happy with this.
To post a comment you must log in.