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

Check time spent on attempting RPC to avoid spending too much time on retrying (#1117) #1131

Merged

Conversation

MyonKeminta
Copy link
Contributor

Cherry-pick #1117 to 6.5 branch

MyonKeminta and others added 2 commits January 24, 2024 14:08
… retrying (tikv#1117)

* Check time spent on attempting RPC to avoid spending too much time on retrying

Signed-off-by: MyonKeminta <[email protected]>

* Handle refreshRegionStore

Signed-off-by: MyonKeminta <[email protected]>

* Add test

Signed-off-by: MyonKeminta <[email protected]>

* Address comments

Signed-off-by: MyonKeminta <[email protected]>

---------

Signed-off-by: MyonKeminta <[email protected]>
Co-authored-by: MyonKeminta <[email protected]>
Signed-off-by: MyonKeminta <[email protected]>
@disksing disksing merged commit 062f4c9 into tikv:tidb-6.5 Jan 24, 2024
7 of 9 checks passed
MyonKeminta added a commit to MyonKeminta/client-go that referenced this pull request Jan 24, 2024
MyonKeminta added a commit to MyonKeminta/client-go that referenced this pull request Jan 24, 2024
… time on retrying (tikv#1117) (tikv#1131)"

This reverts commit 062f4c9.

Signed-off-by: MyonKeminta <[email protected]>
cfzjywxk pushed a commit that referenced this pull request Jan 24, 2024
… time on retrying (#1117) (#1131)" (#1133)

This reverts commit 062f4c9.

Signed-off-by: MyonKeminta <[email protected]>
Co-authored-by: MyonKeminta <[email protected]>
@MyonKeminta MyonKeminta deleted the m/avoid-too-much-rpc-retry-on-timeout-6.5 branch February 19, 2024 08:57
@MyonKeminta MyonKeminta restored the m/avoid-too-much-rpc-retry-on-timeout-6.5 branch February 19, 2024 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants