Skip to content

support find stores with tiflash_mpp label#481

Merged
disksing merged 20 commits into
tikv:masterfrom
guo-shaoge:cloud_test
May 23, 2022
Merged

support find stores with tiflash_mpp label#481
disksing merged 20 commits into
tikv:masterfrom
guo-shaoge:cloud_test

Conversation

@guo-shaoge
Copy link
Copy Markdown
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
Copy Markdown
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
Comment thread internal/locate/region_cache.go Outdated
Comment thread internal/locate/region_cache.go Outdated
Comment thread internal/locate/region_cache.go Outdated
Comment thread internal/locate/region_cache.go
@you06
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

/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