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

txn: attach cluster id in kv request #1208

Merged
merged 2 commits into from
Mar 8, 2024
Merged

txn: attach cluster id in kv request #1208

merged 2 commits into from
Mar 8, 2024

Conversation

cfzjywxk
Copy link
Contributor

@cfzjywxk cfzjywxk commented Mar 7, 2024

Ref: tikv/tikv#16545

Adding cluster id information into the kv requests.

@cfzjywxk cfzjywxk force-pushed the attach_cluster_id branch from 4d70d3f to 369c906 Compare March 7, 2024 12:13
@cfzjywxk
Copy link
Contributor Author

cfzjywxk commented Mar 8, 2024 •
edited
Loading 

go: updates to go.mod needed; to update it:
	go mod tidy
Error: Process completed with exit code 1.

Why the CI keep failing like this? 🤔

@you06
Copy link
Contributor

you06 commented Mar 8, 2024

Why the CI keep failing like this? 🤔

Also need to update integration_tests/go.mod.

@cfzjywxk cfzjywxk force-pushed the attach_cluster_id branch from 369c906 to 8970eff Compare March 8, 2024 05:55
@cfzjywxk
Copy link
Contributor Author

cfzjywxk commented Mar 8, 2024

@you06
Thanks for the reminding.

@cfzjywxk cfzjywxk merged commit 9a37a0a into tikv:master Mar 8, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@you06 you06 you06 approved these changes

@zyguan zyguan zyguan approved these changes

@MyonKeminta MyonKeminta Awaiting requested review from MyonKeminta

@ekexium ekexium Awaiting requested review from ekexium

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cfzjywxk @you06 @zyguan