Skip to content

retry stale read as stale if leader is not accessible#1566

Merged
ti-chi-bot[bot] merged 1 commit intotikv:masterfrom
mittalrishabh:rishabh_retry
Feb 5, 2025
Merged

retry stale read as stale if leader is not accessible#1566
ti-chi-bot[bot] merged 1 commit intotikv:masterfrom
mittalrishabh:rishabh_retry

Conversation

@mittalrishabh
Copy link
Member

issue number : tikv/tikv#17422
It is the extension of the original PR #1509. I have observed that in case of hot regions, leaders are inaccessible and stale reads are retried as follower_follower. I have added a change if leader is not reachable or epoch doesn't mach than retry stale read as stale instead of follower.
github

Signed-off-by: rishabh_mittal <mittalrishabh@gmail.com>
@ti-chi-bot ti-chi-bot bot added dco-signoff: yes Indicates the PR's author has signed the dco. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Feb 3, 2025
@mittalrishabh
Copy link
Member Author

@cfzjywxk please review this PR as original PR was closed #1534

@cfzjywxk cfzjywxk requested review from crazycs520, you06 and zyguan and removed request for crazycs520 February 5, 2025 02:14
@ti-chi-bot ti-chi-bot bot added needs-1-more-lgtm Indicates a PR needs 1 more LGTM. approved labels Feb 5, 2025
@ti-chi-bot
Copy link

ti-chi-bot bot commented Feb 5, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfzjywxk, crazycs520

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [cfzjywxk,crazycs520]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added lgtm and removed needs-1-more-lgtm Indicates a PR needs 1 more LGTM. labels Feb 5, 2025
@ti-chi-bot
Copy link

ti-chi-bot bot commented Feb 5, 2025

[LGTM Timeline notifier]

Timeline:

  • 2025-02-05 02:15:48.868669181 +0000 UTC m=+238020.401081177: ☑️ agreed by cfzjywxk.
  • 2025-02-05 02:58:49.114252018 +0000 UTC m=+240600.646664016: ☑️ agreed by crazycs520.

@ti-chi-bot ti-chi-bot bot merged commit ad5b9e2 into tikv:master Feb 5, 2025
11 checks passed
AvaisP added a commit to AvaisP/client-go that referenced this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved dco-signoff: yes Indicates the PR's author has signed the dco. lgtm size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants