Run remove_push_actions_from_staging in foreground (#8081)

If we got an error persisting an event, we would try to remove the push actions
asynchronously, which would lead to a 'Re-starting finished log context'
warning.

I don't think there's any need for this to be asynchronous.
This commit is contained in:
Richard van der Hoff 2020-08-13 17:05:31 +01:00 committed by GitHub
parent ff0e894656
commit 53834bb9c4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 3 deletions

1
changelog.d/8081.bugfix Normal file
View File

@ -0,0 +1 @@
Fix `Re-starting finished log context PUT-nnnn` warning when event persistence failed.

View File

@ -891,9 +891,7 @@ class EventCreationHandler(object):
except Exception: except Exception:
# Ensure that we actually remove the entries in the push actions # Ensure that we actually remove the entries in the push actions
# staging area, if we calculated them. # staging area, if we calculated them.
run_in_background( await self.store.remove_push_actions_from_staging(event.event_id)
self.store.remove_push_actions_from_staging, event.event_id
)
raise raise
async def _validate_canonical_alias( async def _validate_canonical_alias(