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

Suggested Improvement: Modify the Implementation of eventTrigger to Use a List #605

Open
garfcat opened this issue Sep 27, 2023 · 1 comment

Comments

@garfcat
Copy link

garfcat commented Sep 27, 2023

Suggested Improvement: Modify the Implementation of eventTrigger to Use a List

I would like to propose a change in the implementation of eventTrigger from a map to a list. This modification would allow for the execution of different functions for the same event. Currently, the implementation uses a map, which makes it impossible to perform different actions on the same event. For example, when dealing with the shadowCreateOrUpdateTrigger event, there might be a need to update a cache and also send a message to a message queue. With the current implementation, it would require registering two separate events to achieve this functionality.

By transitioning to a list-based approach, it becomes feasible to execute multiple actions for a single event, providing greater flexibility and efficiency in handling event-triggered tasks.

@miaow999
Copy link
Contributor

miaow999 commented Oct 8, 2023

your suggestion is very good, we will make changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants