-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
- #23650
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
- #23650
Conversation
I think this does not yet belong to Symfony EventDispatcher: |
I beleive @lyrixx is against the proposal as it has a competing implementation in the works. |
If this accepted I don't believe it should be part of the EventDispatcher but has it's own Component. The EventDispatcher Symfony uses is based on the Mediator pattern, where the event can be modified after being dispatched, the Async may not work as one currently excepts from EventDispatcher. In fact, the name EventDispatcher might be wrongfully used for this component 😄 |
I would very like such a feature in the core, but have the same opinion as @sstok that it shouldnt be mixed up with the current event dispatcher, but instead should be its own component with specifically optimized interfaces for this purpose. |
what is the status of this feature ? |
Thank you for this PR. I agree with Nicholas. The event dispatcher component should not be used for async messages like this. When using async messages you want events to be broadcasted and immutable. That are two fundamental features the event dispatcher does not support by design. I was down this route a few year ago with https://github.com/fervo/deferred-event-worker I suggest this PR to be closed. |
hi. Thanks for this PR. I also agree with @Nyholm and @nicolas-grekas We may re-open it later when a nice support of async will land in Symfony. For now, It's a bit too early IMHO. I suggest this PR to be closed. |
Let's close this because of the downvotes from the Core Team members .. but we can reconsider it in the future. Thanks! |
With the new messaging component #24411 , you'll be able to defer the execution of events. |
Uh oh!
There was an error while loading. Please reload this page.