| Commit message (Collapse) | Author | Files | Lines |
|---|
|
We don't do anything with the result, so this is needed to give this code a
logcontext.
|
|
This brings it into line with on_new_notifications and on_new_receipts. It
requires a little bit of hoop-jumping in EmailPusher to load the throttle
params before the first loop.
|
|
`on_new_notifications` and `on_new_receipts` in `HttpPusher` and `EmailPusher`
now always return synchronously, so we can remove the `defer.gatherResults` on
their results, and the `run_as_background_process` wrappers can be removed too
because the PusherPool methods will now complete quickly enough.
|
|
Each pusher has its own loop which runs for as long as it has work to do. This
should run in its own background thread with its own logcontext, as other
similar loops elsewhere in the system do - which means that CPU usage is
consistently attributed to that loop, rather than to whatever request happened
to start the loop.
|
|
simplifies the interface to _start_pushers
|
|
... and use it from start_pusher_by_id. This mostly simplifies
start_pusher_by_id.
|
|
This is public (or at least, called from outside the class), so ought to have a
better name.
|
|
This fallback didn't work, and was removed in #4069.
|
