tikvrpc: add default request origin (#1975)#1977
Conversation
ref pingcap/tidb#68799 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@ekexium This PR has conflicts, I have hold it. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This is an automated cherry-pick of #1975
What changed
tikvrpc.kvrpcpb.Context.request_originfrom that default when creating or attaching request context.Dependency stack
RequestOriginandContext.request_origin.replaceremains; the root module, integration test module, and example modules are bumped directly to the merged kvproto pseudo-version.Tests
gotestsum --format short-verbose -- ./tikvrpcgo test ./tikvrpc -run '^TestDefaultRequestOrigin$' -racecd integration_tests && go list -mod=readonly ./...for mod in $(find examples -name go.mod -print | sort); do (cd "${mod%/go.mod}" && go list -mod=readonly ./...); donegit diff --checkSummary by CodeRabbit
New Features
Chores
Tests