diff options
author | Eric Eastwood <erice@element.io> | 2023-06-13 16:34:54 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-13 16:34:54 -0500 |
commit | 8ddb2de55387d54bac53138f374f55c7608991ce (patch) | |
tree | de00021624bb076021113dbdbcd25d2a5eb76199 /synapse/util/__init__.py | |
parent | Replace `EventContext` fields `prev_group` and `delta_ids` with field `state_... (diff) | |
download | synapse-8ddb2de55387d54bac53138f374f55c7608991ce.tar.xz |
Document `looping_call()` functionality that will wait for the given function to finish before scheduling another (#15772)
Thanks to @erikjohnston for clarifying, https://github.com/matrix-org/synapse/pull/15743#discussion_r1226544457 We don't have to worry about calls stacking up if the given function takes longer than the scheduled time.
Diffstat (limited to 'synapse/util/__init__.py')
-rw-r--r-- | synapse/util/__init__.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/synapse/util/__init__.py b/synapse/util/__init__.py index 7ea0c4c36b..9f3b8741c1 100644 --- a/synapse/util/__init__.py +++ b/synapse/util/__init__.py @@ -116,6 +116,11 @@ class Clock: Waits `msec` initially before calling `f` for the first time. + If the function given to `looping_call` returns an awaitable/deferred, the next + call isn't scheduled until after the returned awaitable has finished. We get + this functionality thanks to this function being a thin wrapper around + `twisted.internet.task.LoopingCall`. + Note that the function will be called with no logcontext, so if it is anything other than trivial, you probably want to wrap it in run_as_background_process. |