-
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
executor,infoschema: move dataForTableConstraints
into executor
#15240
executor,infoschema: move dataForTableConstraints
into executor
#15240
Conversation
… to `executor` Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Thanks for your contribution. If your PR get merged, you will be rewarded 50 points. |
Codecov Report
@@ Coverage Diff @@
## master #15240 +/- ##
=========================================
Coverage 80.395% 80.395%
=========================================
Files 503 503
Lines 133589 133589
=========================================
Hits 107399 107399
Misses 17778 17778
Partials 8412 8412 |
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.
Rest LGTM
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
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.
LGTM
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.
lgtm
/run-all-tests |
Team Rustin-Liu complete task #15027 and get 50 score, currerent score 100 |
UCP #15027
Signed-off-by: Rustin-Liu rustin.liu@gmail.com
What problem does this PR solve?
move
dataForTableConstraints
into executor.What is changed and how it works?
move
dataForTableConstraints
into executor.Check List
Tests
Code changes
None
Side effects
None
Related changes
None
Release note
None