Skip to content

support choosing non-pending tiflash peer when buildBatchCopTask#144

Merged
gengliqi merged 8 commits intotikv:masterfrom
guo-shaoge:tiflash_pending_peer
May 31, 2023
Merged

support choosing non-pending tiflash peer when buildBatchCopTask#144
gengliqi merged 8 commits intotikv:masterfrom
guo-shaoge:tiflash_pending_peer

Conversation

@guo-shaoge
Copy link
Contributor

@guo-shaoge guo-shaoge commented May 28, 2023

issue: pingcap/tiflash#7301

Before: Task can dispatch to store whose region is still pending.
After: Try not to dispatch task to pending store.

What's changed:

  1. Change function declaration of getRegionByKey() and getRegionByID(), return whole response instead of only region info.
  2. Record pending store of region in region cache.
  3. Try dispatch task to non_pending_store of region when buildBatchCopTasks.

Signed-off-by: guo-shaoge <shaoge1994@163.com>
@guo-shaoge guo-shaoge force-pushed the tiflash_pending_peer branch from 623154a to b48b0a7 Compare May 28, 2023 14:22
…pending_peer

Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Copy link
Contributor

@JaySon-Huang JaySon-Huang left a comment

Choose a reason for hiding this comment

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

Rest LGTM

Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
@gengliqi
Copy link
Member

/merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants