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

mpp: support returning regions that need retry #751

Merged
merged 4 commits into from
Apr 29, 2021
Merged

mpp: support returning regions that need retry #751

merged 4 commits into from
Apr 29, 2021

Conversation

hanfei1991
Copy link
Member

Mpp need to refresh the TiDB client's cache when there are region retry in TiFlash. TiFlash returns these region info and TiDB refresh the cache.

@fzhedu
Copy link
Contributor

fzhedu commented Apr 13, 2021

/LGTM

@fzhedu
Copy link
Contributor

fzhedu commented Apr 26, 2021

/merge

@CLAassistant
Copy link

CLAassistant commented Apr 26, 2021 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@fzhedu
Copy link
Contributor

fzhedu commented Apr 26, 2021

/merge

windtalker
windtalker approved these changes Apr 29, 2021
View reviewed changes
Copy link
Contributor

@windtalker windtalker left a comment

Choose a reason for hiding this comment

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

LGTM

@fzhedu
Copy link
Contributor

fzhedu commented Apr 29, 2021

/LGTM

@fzhedu
Copy link
Contributor

fzhedu commented Apr 29, 2021

/merge

fzhedu
fzhedu approved these changes Apr 29, 2021
View reviewed changes
Copy link
Contributor

@fzhedu fzhedu left a comment

Choose a reason for hiding this comment

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

LGTM

@hanfei1991 hanfei1991 merged commit 65f54a2 into pingcap:master Apr 29, 2021
@hanfei1991 hanfei1991 deleted the hanfei/region-errors branch April 29, 2021 09:38
This was referenced May 3, 2021
Merged
Merged
@ti-srebot ti-srebot mentioned this pull request May 6, 2021
Merged
windtalker pushed a commit to windtalker/kvproto that referenced this pull request May 7, 2021
@hanfei1991 @fzhedu @windtalker
mpp: support returning regions that need retry (pingcap#751)
76b14e4 
* mpp: support return of region errors

* return region list

Co-authored-by: Zhuhe Fang <fzhedu@gmail.com>
@ti-srebot ti-srebot mentioned this pull request May 8, 2021
Merged
hanfei1991 added a commit to hanfei1991/kvproto that referenced this pull request May 13, 2021
@hanfei1991 @fzhedu
mpp: support returning regions that need retry (pingcap#751)
8613d16 
* mpp: support return of region errors

* return region list

Co-authored-by: Zhuhe Fang <fzhedu@gmail.com>
hanfei1991 added a commit that referenced this pull request May 13, 2021
@hanfei1991 @fzhedu
mpp: support returning regions that need retry (#751) (#763)
12df7bb 
* mpp: support returning regions that need retry (#751)

* mpp: support return of region errors

* return region list

Co-authored-by: Zhuhe Fang <fzhedu@gmail.com>

* add retry regions for batch cop response (#761)

Co-authored-by: Zhuhe Fang <fzhedu@gmail.com>
daimashusheng pushed a commit to daimashusheng/kvproto that referenced this pull request Sep 2, 2021
@hanfei1991 @fzhedu
mpp: support returning regions that need retry (pingcap#751)
1eeaa57 
* mpp: support return of region errors

* return region list

Co-authored-by: Zhuhe Fang <fzhedu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fzhedu fzhedu fzhedu approved these changes

@windtalker windtalker windtalker approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hanfei1991 @fzhedu @CLAassistant @windtalker