Skip to content

Try other peer when current leader of this region is not available. #232

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
Merged
Little-Wallace merged 13 commits into from
Jul 15, 2021
Merged

Try other peer when current leader of this region is not available. #232

Little-Wallace merged 13 commits into from
Jul 15, 2021

Conversation

@Little-Wallace
Copy link
Contributor

@Little-Wallace Little-Wallace commented Jul 12, 2021

What problem does this PR solve?

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Code changes

  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation
  • Need to be included in the release note

@Little-Wallace
refactor region cache
45f2e81 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
refactor proxy
2b3f57e 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
rename
06c6442 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
check region before forward
7cd830f 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
support try other peer as leader
368bcd3 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
fix strategy
6283554 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
remove sync
0271d14 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
optimize strategy
cea2131 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
makr forward again after check health success
05c26ff 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
remove unuse code
0e6c171 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
fix metrics
608b6b9 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
fix strategy
29fb387 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
@Little-Wallace
fix bug
a15a55d 
Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
birdstorm
birdstorm approved these changes Jul 14, 2021
View reviewed changes
Copy link
Collaborator

@birdstorm birdstorm left a comment

Choose a reason for hiding this comment

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

LGTM

@Little-Wallace Little-Wallace merged commit d4d1c6a into tikv:master Jul 15, 2021
ti-srebot pushed a commit to ti-srebot/client-java that referenced this pull request Jul 15, 2021
@Little-Wallace @ti-srebot
cherry pick tikv#232 to release-3.1
83a9a94 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Jul 15, 2021
Merged
@ti-srebot
Copy link
Collaborator

ti-srebot commented Jul 15, 2021

cherry pick to release-3.1 in PR #234

birdstorm pushed a commit that referenced this pull request Jul 18, 2021
@ti-srebot @Little-Wallace
Try other peer when current leader of this region is not available. (#…
d0b4965 
…232) (#234)

* cherry pick #232 to release-3.1

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

* fix bug

Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>

Co-authored-by: Wallace <bupt2013211450@gmail.com>
@Little-Wallace Little-Wallace deleted the proxy branch July 18, 2021 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@birdstorm birdstorm birdstorm approved these changes

Assignees

No one assigned

Labels

status/LGT1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Little-Wallace @ti-srebot @birdstorm