Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[fix #389] move CircuitBreaker to TiSession (#390) - #391

Merged
marsishandsome merged 5 commits into
tikv:release-3.1from
ti-srebot:release-3.1-faf1da74bf00
Dec 11, 2021
Merged

[fix #389] move CircuitBreaker to TiSession (#390)#391
marsishandsome merged 5 commits into
tikv:release-3.1from
ti-srebot:release-3.1-faf1da74bf00

Conversation

@ti-srebot

@ti-srebot ti-srebot commented Dec 11, 2021

Copy link
Copy Markdown
Collaborator

cherry-pick #390 to release-3.1
You can switch your code base to this Pull Request by using git-extras:

# In client-java repo:
git pr https://github.com/tikv/client-java/pull/391

After apply modifications, you can push your change to this PR via:

git push [email protected]:ti-srebot/client-java.git pr/391:release-3.1-faf1da74bf00

Signed-off-by: Jian Zhang [email protected]

What problem does this PR solve?

What is changed and how it works?

make CircuitBreaker been created for every TiSession instead of every SmartRawKVClient

Check List

Tests

  • Unit test

Related changes

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

@ti-srebot

Copy link
Copy Markdown
Collaborator Author

/run-all-tests

@marsishandsome

Copy link
Copy Markdown
Collaborator

/run-all-tests

@marsishandsome

Copy link
Copy Markdown
Collaborator

/run-all-tests

marsishandsome
marsishandsome previously approved these changes Dec 11, 2021

@marsishandsome marsishandsome left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: marsishandsome <[email protected]>
@marsishandsome

Copy link
Copy Markdown
Collaborator

/run-all-tests

@marsishandsome marsishandsome left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@marsishandsome
marsishandsome merged commit 83fb5f4 into tikv:release-3.1 Dec 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants