Merge lp:~stub/launchpad/bug-658124-revision-karma into lp:launchpad
Proposed by
Stuart Bishop
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Robert Collins | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 11694 | ||||
Proposed branch: | lp:~stub/launchpad/bug-658124-revision-karma | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
83 lines (+13/-14) 3 files modified
lib/lp/code/interfaces/revision.py (+1/-1) lib/lp/code/model/revision.py (+11/-12) lib/lp/code/scripts/revisionkarma.py (+1/-1) |
||||
To merge this branch: | bzr merge lp:~stub/launchpad/bug-658124-revision-karma | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | Approve | ||
Robert Collins (community) | release-critical | Approve | |
Review via email: mp+38181@code.launchpad.net |
Commit message
Fix performance of revision karma allocator with PostgreSQL 8.4
Description of the change
The revision karma allocator is currently disabled due to major performance problems with PostgreSQL 8.4, per Bug #658124
To post a comment you must log in.
Looks to me like a DISTINCT ON would be more efficient and faster. Perhaps if you're moving it to the garbo post release it could be reorganised too?