server,resourcemanager: route metadata writes via PD#10260
server,resourcemanager: route metadata writes via PD#10260okJiang wants to merge 2 commits intotikv:masterfrom
Conversation
Signed-off-by: okjiang <819421878@qq.com>
Signed-off-by: okjiang <819421878@qq.com>
|
Skipping CI for Draft Pull Request. |
|
[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 |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the 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 |
What problem does this PR solve?
After watcher scaffold, metadata write traffic should be handled by PD while RM focuses on token/read serving and tracks metadata via watcher updates.
Issue Number: ref #9737
What is changed and how does it work?
This branch is intended as PR6 and functionally depends on #10259.
Because the head branch is in a fork and I cannot push the base branch to
tikv/pd, GitHub cannot create a stacked PR with basecodex/rm-pr4-metadata-watcher-scaffoldin upstream. So this draft currently includes PR4 commit(s) in history as dependency.Behavior changes in this PR layer:
FailedPrecondition(must write via PD)Check List
Tests
Code changes
Side effects
Related changes
Release note