-
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
ddl: Preparation for add index acceleration #36085
ddl: Preparation for add index acceleration #36085
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 unit-test |
942576d
to
fc1d005
Compare
Code Coverage Details: https://codecov.io/github/pingcap/tidb/commit/53b8756e5a07b9a52d594de51f9bc9eb7a28c333 |
/run-check_dev_2 |
/run check_dev_2 |
/run-check-dev |
/run-build |
/run-mysql-test |
/run-chech_dev_2 |
/run-check_dev_2 |
1 similar comment
/run-check_dev_2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found this PR copy some files to solve import cycle
problem. I think it's not a good practice. In this way, we need to maintain two copies of the same code.
4ed247b
to
0b815e1
Compare
0b815e1
to
1b5d9b8
Compare
I have refined the code. |
40b04e6
to
af340d0
Compare
Please resolve the conflicts. |
1f359cf
to
d3a6ed9
Compare
/run-integration-br-test |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: d3a6ed9
|
/run-integration-br-test |
TiDB MergeCI notify✅ Well Done! New fixed [1] after this pr merged.
|
What problem does this PR solve?
Issue Number: ref #35983
Problem Summary:
Do some br/lightning code refactor to solve import cycle problem.
What is changed and how it works?
Check List
Tests