Merge lp://staging/~allenap/launchpad/checkwatches-bulk-reload-bug-572211 into lp://staging/launchpad
Proposed by
Gavin Panella
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Gavin Panella | ||||
| Approved revision: | no longer in the source branch. | ||||
| Merged at revision: | 10892 | ||||
| Proposed branch: | lp://staging/~allenap/launchpad/checkwatches-bulk-reload-bug-572211 | ||||
| Merge into: | lp://staging/launchpad | ||||
| Prerequisite: | lp://staging/~allenap/launchpad/storm-bulk-reload-bug-572211 | ||||
| Diff against target: |
668 lines (+594/-4) 3 files modified
lib/lp/bugs/scripts/checkwatches/core.py (+5/-0) lib/lp/bugs/scripts/tests/test_bugimport.py (+9/-4) lib/lp/code/model/tests/test_sourcepackagerecipe.py (+580/-0) |
||||
| To merge this branch: | bzr merge lp://staging/~allenap/launchpad/checkwatches-bulk-reload-bug-572211 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Abel Deuring (community) | code | Approve | |
|
Review via email:
|
|||
Commit message
Use bulk.reload() to improve the performance of checkwatches.
Description of the change
This uses the new bulk.reload() method to improve the performance of checkwatches. See the linked bug and the merge proposal for the prerequisite branch for an explanation how.
To post a comment you must log in.
Revision history for this message
| Abel Deuring (adeuring) | # |
review:
Approve
(code)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
