Deprecate asynchronous `SchedulerPlugin.{add|remove}_worker` hooks
Open
deprecation
- Dominant language
- Python
- Stars
- 1.7k
- Forks
- 778
- Avg merge
- 2h 50m
- Merged PRs (30d)
- 3
Description
Allowing synchronous and asynchronous `SchedulerPlugin.{add|remove}_worker` hooks makes it harder to reason about their execution order (related: #8486). I'm unaware of asynchronous hooks used in practice, so I propose deprecating these. Should there be any legitimate use cases, we can introduce an asynchronous `after_{add|remove}_worker` hook that would allow strict ordering.
Contributor guide
Assessment
This issue has not been assessed yet.