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
br: fix the issue that config not paused with ttl correctly #31725
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. |
/run-integration-br-tests |
93e9877
to
40029c0
Compare
/run-integration-br-tests |
Code Coverage Details: https://codecov.io/github/pingcap/tidb/commit/37cb9379bb420e41d8a911a486f6753471e7026f |
/run-integration-br-tests |
/run-integration-br-tests |
2 similar comments
/run-integration-br-tests |
/run-integration-br-tests |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 3145ded075257217d5f95946e3e7a5f7f922eb84
|
/hold |
b074c91
to
39f2be4
Compare
/unhold |
5c5d642
to
be41a5a
Compare
/merge |
This pull request has been accepted and is ready to merge. Commit hash: c7d6295
|
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-6.0 in PR #34524 |
TiDB MergeCI notify🔴 Bad News! [1] CI still failing after this pr merged.
|
What problem does this PR solve?
Issue Number: close #31733
Problem Summary:
Currently pause configs not work. it changed the config without ttl.
This PR tried to make ttl works.
What is changed and how it works?
use the correct url.
Check List
Tests
Side effects
Documentation
Release note