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

Improve EventBus experience #625

Open
zhenlei520 opened this issue May 24, 2023 · 1 comment
Open

Improve EventBus experience #625

zhenlei520 opened this issue May 24, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@zhenlei520
Copy link
Contributor

Description

Currently using ISagaEventHandler, if not specified separately, the value of FailureLevels of the HandleAsync method is FailureLevels.Throw, which causes the CancelHandler corresponding to the current Handler to not be executed when an exception occurs in the Handler, but we are more accustomed to thinking that Handler After an exception occurs, the corresponding CancelHandler should be executed

.NET version

.NET 6.0

MASA Framework version

1.0.0-rc.2

@zhenlei520
Copy link
Contributor Author

When using ISagaEventHandler, we are unlikely to use the function of Handler orchestration. At this time, only one Handler and one CancelHandler are needed. When the Handler is abnormal, the corresponding CancelHandler will be executed. This is a habit, but if you do not use orchestration, CancelHandler needs to be considered It will not cause data errors due to compensation (the framework itself does not know which step of the Handler execution is wrong), and choosing to use event orchestration is a better choice

@zhenlei520 zhenlei520 self-assigned this May 24, 2023
@duiapro duiapro added the enhancement New feature or request label Apr 3, 2024
@duiapro duiapro self-assigned this Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants