-
Notifications
You must be signed in to change notification settings - Fork 876
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
chore(transaction): Untie scheduling from multi status #2590
Conversation
Signed-off-by: Vladislav Oleshko <vlad@dragonflydb.io>
423924b
to
508830e
Compare
Signed-off-by: Vladislav Oleshko <vlad@dragonflydb.io>
595ab7a
to
86800eb
Compare
Can you please expand in the PR description what is the potential ability you unlock with this PR? |
Can you please expand in the PR description what is the potential ability you unlock with this PR?
Goal: Being able to use ScheduleSingleHop()/Schedule() as a starter for multi transactions, and thus later allow single hop multi transactions |
29b9d55
to
3138286
Compare
Signed-off-by: Vladislav Oleshko <vlad@dragonflydb.io>
3138286
to
b957cd6
Compare
) * chore(transaction): Untie scheduling from multi status Idea: We decide whether we have to schedule not based on our multi status (atomic multi), but solely based on the fact if COORD_SCHED is set Goal: Being able to use ScheduleSingleHop()/Schedule() for multi transactions, and thus later allow single hop multi transactions --------- Signed-off-by: Vladislav Oleshko <vlad@dragonflydb.io>
Moves part out of #2376
Idea: We decide whether we have to schedule not based on our multi status (atomic multi), but solely based on the fact if COORD_SCHED is set
Goal: Being able to use ScheduleSingleHop()/Schedule() for multi transactions, and thus later allow single hop multi transactions