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.

Sign up for GitHub

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

Jump to bottom

Add routing hook in BaseShardingEngine #2273

Closed
cherrylzhao opened this issue Apr 26, 2019 · 0 comments
Closed

Add routing hook in BaseShardingEngine #2273

cherrylzhao opened this issue Apr 26, 2019 · 0 comments
Assignees
@cherrylzhao
Labels
feature: sharding type: enhancement
Milestone
4.0.0-RC2

Comments

@cherrylzhao
Copy link
Member

cherrylzhao commented Apr 26, 2019 •
edited
Loading

transaction-base-saga need RouteResult to do sql-revert.
so we can add a routing hook within BaseShardingEngine to pass these context into saga module.
@cherrylzhao cherrylzhao changed the title Add ShardHook in BaseShardingEngine Add rewrite hook in BaseShardingEngine Apr 26, 2019
@cherrylzhao cherrylzhao changed the title Add rewrite hook in BaseShardingEngine Add route hook in BaseShardingEngine Apr 26, 2019
@cherrylzhao cherrylzhao changed the title Add route hook in BaseShardingEngine Add routing hook in BaseShardingEngine Apr 26, 2019
@cherrylzhao cherrylzhao self-assigned this Apr 26, 2019
@cherrylzhao cherrylzhao modified the milestone: 4.0.0.M2 Apr 26, 2019
terrymanu added a commit that referenced this issue Apr 26, 2019
@terrymanu
Merge pull request #2274 from cherrylzhao/dev
0d2b931 
for #2273 Add ShardHook in BaseShardingEngine
@cherrylzhao cherrylzhao added this to the 4.0.0.M2 milestone Apr 29, 2019
@Harper0122 Harper0122 mentioned this issue May 13, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cherrylzhao cherrylzhao

Labels
feature: sharding type: enhancement
Projects
None yet
Milestone
4.0.0-RC2
Development

No branches or pull requests
1 participant
@cherrylzhao