Skip to content

Replace stale-read traffic with local-remote traffic#877

Closed
you06 wants to merge 3 commits intotikv:tidb-6.5from
you06:cross-az-traffic
Closed

Replace stale-read traffic with local-remote traffic#877
you06 wants to merge 3 commits intotikv:tidb-6.5from
you06:cross-az-traffic

Conversation

@you06
Copy link
Contributor

@you06 you06 commented Jul 10, 2023

By recording stale-read-miss traffic, we can know the traffic of remote reading.

But in some cases, the replica selector just skip the unavailable local replica and do a remote stale read. Even such stale-read is successful, billed traffic may be produced and need to be specified.

This PR will record the traffic of stale-read requests by local and remote types when there is labels. In the example metrics, the local store is in restart progress, but the stale-read success rate is still high, and we can find the remote traffic with this PR.

image

you06 added 3 commits July 10, 2023 17:36
Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: you06 <you1474600@gmail.com>
@cfzjywxk
Copy link
Contributor

@crazycs520 PTAL

@cfzjywxk
Copy link
Contributor

@you06 Is this included in #878?

@you06
Copy link
Contributor Author

you06 commented Jul 26, 2023

@you06 Is this included in #878?

Yeah, this PR can be closed.

@you06 you06 closed this Jul 26, 2023
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.

2 participants