Skip to content

[to #375] use scanRegions request to warm up client (#383)#438

Merged
zz-jason merged 4 commits into
tikv:release-3.1from
ti-srebot:release-3.1-468e999c2b97
Dec 30, 2021
Merged

[to #375] use scanRegions request to warm up client (#383)#438
zz-jason merged 4 commits into
tikv:release-3.1from
ti-srebot:release-3.1-468e999c2b97

Conversation

@ti-srebot

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

Copy link
Copy Markdown
Collaborator

cherry-pick #383 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/438

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

git push git@github.com:ti-srebot/client-java.git pr/438:release-3.1-468e999c2b97

use scanRegions request with limit to warm up client.

new configurations:

tikv.warm_up.enable default=true
tikv.grpc.warm_up_timeout_in_ms default=5000ms

@ti-srebot

Copy link
Copy Markdown
Collaborator Author

/run-all-tests

Signed-off-by: birdstorm <samuelwyf@hotmail.com>
Signed-off-by: birdstorm <samuelwyf@hotmail.com>
@birdstorm birdstorm force-pushed the release-3.1-468e999c2b97 branch from 28de8f1 to ce5b016 Compare December 28, 2021 07:23
@birdstorm

Copy link
Copy Markdown
Collaborator

/run-all-tests

birdstorm
birdstorm previously approved these changes Dec 29, 2021

@birdstorm birdstorm 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: birdstorm <samuelwyf@hotmail.com>

@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

@zz-jason

Copy link
Copy Markdown
Member

/merge

@ti-srebot

Copy link
Copy Markdown
Collaborator Author

/run-all-tests

@zz-jason zz-jason merged commit fdb0d74 into tikv:release-3.1 Dec 30, 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.

4 participants