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

Cherry pick some patches to 7.1 #940

Merged
merged 9 commits into from
Sep 26, 2023
Merged

Cherry pick some patches to 7.1 #940

merged 9 commits into from
Sep 26, 2023

Conversation

you06
Copy link
Contributor

@you06 you06 commented Aug 11, 2023 •
edited
Loading

Cherry pick #843, #923, #911, #942 to 7.1 branch.

  • Fix the inbalance issue of replica read request when store is down and up by reloading the latest region in background.
  • For replica read, when leader is busy, fallback to follower.
  • Set max retry for replica read.
  • Add test of replica selector's state for stale read.

@you06 you06 mentioned this pull request Aug 14, 2023
Closed
@you06 you06 changed the title reload region cache when store is resolved from invalid status (#843) Cherry pick some patches to 7.1 Aug 14, 2023
you06 and others added 6 commits August 15, 2023 15:58
@you06 @disksing
reload region cache when store is resolved from invalid status (tikv#843
60bbaf1 
)

Signed-off-by: you06 <you1474600@gmail.com>
Co-authored-by: disksing <i@disksing.com>
@you06 @cfzjywxk
fallback to follower when leader is busy (tikv#916) (tikv#923)
56e4ad6 
* fallback to follower when leader is busy

Signed-off-by: you06 <you1474600@gmail.com>
Co-authored-by: cfzjywxk <cfzjywxk@gmail.com>
Co-authored-by: cfzjywxk <lsswxrxr@163.com>
@cfzjywxk @you06
Resume max retry time check for stale read retry with leader option(t…
c2e8548 
…ikv#903) (tikv#911)

* Resume max retry time check for stale read retry with leader option

Signed-off-by: cfzjywxk <lsswxrxr@163.com>

* add cancel

Signed-off-by: cfzjywxk <lsswxrxr@163.com>

---------

Signed-off-by: cfzjywxk <lsswxrxr@163.com>
@you06
add region cache state test & fix some issues of replica selector (ti…
318f84d 
…kv#910)

Signed-off-by: you06 <you1474600@gmail.com>

remove duplicate code

Signed-off-by: you06 <you1474600@gmail.com>
@you06
enable workflow for tidb-7.1
a1ee1da 
Signed-off-by: you06 <you1474600@gmail.com>
@you06
update
233bf4f 
Signed-off-by: you06 <you1474600@gmail.com>

update

Signed-off-by: you06 <you1474600@gmail.com>

fix test

Signed-off-by: you06 <you1474600@gmail.com>

fix test

Signed-off-by: you06 <you1474600@gmail.com>
@you06
lint
b29a82a 
Signed-off-by: you06 <you1474600@gmail.com>
@you06
lint
f087d39 
Signed-off-by: you06 <you1474600@gmail.com>
@you06
fix flaky test
38e1319 
Signed-off-by: you06 <you1474600@gmail.com>
@cfzjywxk
Copy link
Contributor

/cc @crazycs520

crazycs520
crazycs520 approved these changes Sep 26, 2023
View reviewed changes
Copy link
Contributor

@crazycs520 crazycs520 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cfzjywxk cfzjywxk merged commit 83165bd into tikv:tidb-7.1 Sep 26, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crazycs520 crazycs520 crazycs520 approved these changes

@cfzjywxk cfzjywxk cfzjywxk approved these changes

@zyguan zyguan Awaiting requested review from zyguan

@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
@you06 @cfzjywxk @crazycs520