Conversation
Copy link
Copy Markdown
Member
This does two things:
- Respect the rather new
Defaults.run_asyncwhen deciding ifupdate_persistenceshould be called or not. This was just overlooked by me in Add Defaults.run_async support #2210 - call
update_persistenceonly once per update and only if the update was handled by at least once sync handler. Having a update handled by multiple groups can slow down the bot significantly otherwise.
Also this actually documents how stuff is updated (and fixes the annotation of process_update on the fly)
(cherry picked from commit 89c522d )
Copy link
Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM