Merge lp://qastaging/~ken-vandine/gwibber/lp_906916 into lp://qastaging/gwibber

Proposed by Ken VanDine
Status: Merged
Merged at revision: 1235
Proposed branch: lp://qastaging/~ken-vandine/gwibber/lp_906916
Merge into: lp://qastaging/gwibber
Diff against target: 77 lines (+12/-25)
1 file modified
gwibber/microblog/dispatcher.py (+12/-25)
To merge this branch: bzr merge lp://qastaging/~ken-vandine/gwibber/lp_906916
Reviewer Review Type Date Requested Status
Michael Terry (community) Approve
Review via email: mp+88455@code.qastaging.launchpad.net

Description of the change

Drop the use of threading.Thread just to create a multiprocessing pool and ensure the pool gets closed after operations are done. This prevents the constant polling between workers which caused frequent wakeups and excessive power use.

To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

Looks great!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.