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.

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

support find stores with tiflash_mpp label #481

Merged
merged 20 commits into from
May 23, 2022
Merged

Conversation

guo-shaoge
Copy link
Contributor

@guo-shaoge guo-shaoge commented Apr 27, 2022

Signed-off-by: guo-shaoge shaoge1994@163.com

close: #498

Related tidb pr: pingcap/tidb#33535

  1. add GetTiFlashMPPRPCContextByConsistentHash and its related functions.
  2. handle tiflash_mpp in onSendFail().
  3. change some if branch from ctx.Store.storeType == tikvrpc.TiFlash to ctx.Store.storeType.IsTiFlashRelatedType().

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

/hold

Signed-off-by: guo-shaoge <shaoge1994@163.com>
@guo-shaoge guo-shaoge changed the title [DNM] support find stores with tiflash_mpp label support find stores with tiflash_mpp label May 16, 2022
@you06
Copy link
Contributor

you06 commented May 18, 2022

What about adding counter and duration metrics for the cache invalidation and loading.

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

guo-shaoge commented May 19, 2022

What about adding counter and duration metrics for the cache invalidation and loading.

To keep this pr clean, I will update metrics in the next pr(#509)~

Also InvalidateTiFlashMPPStoresIfGRPCError() is added, so the cache will only be invalidated if error is grpc.Unavailable(e.g., tiflash node is down etc.). This can reduce the chance of cache invalidation.

@guo-shaoge guo-shaoge requested a review from you06 May 19, 2022 07:15
sticnarf and others added 4 commits May 20, 2022 14:44
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
@XuHuaiyu
Copy link

/merge

guo-shaoge and others added 3 commits May 20, 2022 17:16
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: Yilin Chen <sticnarf@gmail.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
guo-shaoge and others added 5 commits May 23, 2022 11:02
Signed-off-by: guo-shaoge <shaoge1994@163.com>
Signed-off-by: Yilin Chen <sticnarf@gmail.com>
Signed-off-by: guo-shaoge <shaoge1994@163.com>
@disksing disksing merged commit 700cbe6 into tikv:master May 23, 2022
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.

5 participants