-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
server: let plan replayer support forward request #29299
Conversation
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
Signed-off-by: yisaer <[email protected]> add log Signed-off-by: yisaer <[email protected]> add log Signed-off-by: yisaer <[email protected]> add log Signed-off-by: yisaer <[email protected]> add log Signed-off-by: yisaer <[email protected]> fix get topo Signed-off-by: yisaer <[email protected]> revise log Signed-off-by: yisaer <[email protected]> fix lint Signed-off-by: yisaer <[email protected]> fix lint Signed-off-by: yisaer <[email protected]> fix lint Signed-off-by: yisaer <[email protected]> revise handle error Signed-off-by: yisaer <[email protected]> revise handle error Signed-off-by: yisaer <[email protected]>
bc473df
to
29e164e
Compare
Signed-off-by: yisaer <[email protected]>
Signed-off-by: yisaer <[email protected]>
Signed-off-by: yisaer <[email protected]>
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 116501c
|
Signed-off-by: ti-srebot <[email protected]>
cherry pick to release-5.3 in PR #29436 |
/run-cherry-picker |
What problem does this PR solve?
Issue Number: close #29495
Problem Summary:
Currently,
plan replayer
dump api need the tidb-server which handle the request should be the same tidb-server which also receive the plan replayer statement. This will be hard to use in reality as most users exposes tidb status address behind the proxy (like kubernetes).This request makes plan replayer able to forward request if the both tidb-servers are not the same one.
What is changed and how it works?
Check List
Tests
I deploy a tidbcluster with 1pd/1kv/2db, I execute
plan replayer
statement to tidb-1, and try to get file from both tidb-1 and tidb-2, both requests are successful.Release note