Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[Bug]: sync txn request does not execute in the order of child request #498

Closed
bsbds opened this issue Nov 2, 2023 · 0 comments
Closed

[Bug]: sync txn request does not execute in the order of child request #498

bsbds opened this issue Nov 2, 2023 · 0 comments
Assignees
@bsbds
Labels
bug Something isn't working
Milestone
v0.7.0

Comments

@bsbds
Copy link
Collaborator

bsbds commented Nov 2, 2023

In etcd the child requests in a single txn is execute in order. However current sync_txn_request implementation uses BFS(Breadth-first search
) to process the txn request and it is not suitable for nested txns. We need to refactor this to use DFS.
@bsbds bsbds mentioned this issue Nov 3, 2023
Closed
@Phoenix500526 Phoenix500526 added the bug Something isn't working label Nov 10, 2023
@bsbds bsbds added this to the v0.7.0 milestone Nov 22, 2023
@bsbds bsbds closed this as completed Aug 27, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@bsbds bsbds

Labels
bug Something isn't working
Projects
None yet
Milestone
v0.7.0
Development

Successfully merging a pull request may close this issue.

refactor: replace BFS with DFS in sync_txn_request bsbds/Xline
2 participants
@Phoenix500526 @bsbds