Skip to content

ruv2: align bypass with request info#1

Closed
disksing wants to merge 1 commit intofeature/ru-scale-default-1.40from
disksing/ruv2-bypass
Closed

ruv2: align bypass with request info#1
disksing wants to merge 1 commit intofeature/ru-scale-default-1.40from
disksing/ruv2-bypass

Conversation

@disksing
Copy link
Copy Markdown

Summary

  • bypass TiKV RU v2 updates only for requests that are already marked as bypassed by RequestInfo
  • reuse the same bypass decision in unary, async, and coprocessor stream response paths
  • add regression coverage for async requests, coprocessor streams, and the bypass decision itself

Why

RU v2 bypass conditions had become broader than the existing resource control bypass rules by also skipping background requests. This change aligns the RU v2 bypass behavior with the existing V1-style request-info bypass semantics.

Testing

  • go test ./internal/client -run 'TestBypassRUV2FollowsRequestInfoBypass|TestSendRequestAsyncUpdateTiKVRUV2'
  • go test ./tikvrpc -run TestCopStreamResponseRecvBypassRUV2
  • go test ./internal/resourcecontrol -run TestMakeRequestInfo

Notes

  • This PR targets feature/ru-scale-default-1.40 because the branch is stacked on top of that change.

Signed-off-by: disksing <i@disksing.com>
@disksing disksing closed this Mar 30, 2026
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.

1 participant