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

consider about adding StreamBasedSaga instead of GraphBasedSaga #43

Open
cherrylzhao opened this issue Apr 26, 2019 · 0 comments
Open
Assignees
Labels
enhancement New feature or request saga base saga transaction

Comments

@cherrylzhao
Copy link
Contributor

currently, we integrated with saga using graph based engine.
it need us to collect all participated actual SQL, then submit to saga actuator in commit/rollback phase. if application crashed before invoking saga actuator, undo log of branch transaction SQL will not be saved, so recovery thread will not be executed correctly.

it's better that encapsulating every actual SQL as a saga task, then submit to saga actuator realtime instead of batch processing all the SQLs at commit/rollback phase. this architecture will make the boundary more clear between shardingsphre and saga, currently we have done some additional work for integrating saga.

@cherrylzhao cherrylzhao self-assigned this Apr 26, 2019
@cherrylzhao cherrylzhao added enhancement New feature or request saga base saga transaction labels Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request saga base saga transaction
Projects
None yet
Development

No branches or pull requests

1 participant