tikvrpc: add default request origin (#1975) - #1978
Conversation
|
@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 |
48392d0 to
992d13d
Compare
ref pingcap/tidb#68799 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io> Signed-off-by: Ziqian Qin <eke@fastmail.com>
992d13d to
00e56d4
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cfzjywxk, zyguan The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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