Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AsyncNotificationCenter asynchronous notifying #4851

Open
aleks-f opened this issue Jan 15, 2025 · 0 comments
Open

AsyncNotificationCenter asynchronous notifying #4851

aleks-f opened this issue Jan 15, 2025 · 0 comments
Assignees
Labels

Comments

@aleks-f
Copy link
Member

aleks-f commented Jan 15, 2025

Is your feature request related to a problem? Please describe.
AsyncNotificationCenter currently receives notifications asynchronously. However, it notifies observers synchronously, which means that processing is as fast as the slowest handler; much worse, a blocked observer notification handler stops all subsequent pending notifications.

Describe the solution you'd like
Add capability to notify asynchronously. This is best added as a property of Notification (or introducing AsyncNotification class as a child of Notification). AsyncNotificationCenter can then use ActiveMethod for spawning notifications marked as asynchronous and keep the list of pending results for housekeeping. Sync notification mode can remain default, so that the current behavior is not broken.

Describe alternatives you've considered
None, AsyncNotificationCenter is async/sync asymmetric and that should be changed.

@aleks-f aleks-f added this to the Release 1.15.0 milestone Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants