Merge lp:~thumper/launchpad/revision-karma-fix into lp:launchpad
Proposed by
Tim Penhey
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | not available | ||||
Proposed branch: | lp:~thumper/launchpad/revision-karma-fix | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
37 lines (+13/-7) 1 file modified
lib/lp/code/scripts/revisionkarma.py (+13/-7) |
||||
To merge this branch: | bzr merge lp:~thumper/launchpad/revision-karma-fix | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Hudson-Doyle | Approve | ||
Review via email: mp+16890@code.launchpad.net |
Commit message
Break out of the loop if karma is ever not allocated.
To post a comment you must log in.
The count added before is too inefficient so removed it.
Changed the looping logic, and stop if we ever fail to allocate karma as it will infinite-loop otherwise.
If it does die, it gives us the revision_id from which to start debugging.