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: align behavior of foreign keys in "SHOW CREATE TABLE" with MySQL. | tidb-test=pr/2312 #51294
executor: align behavior of foreign keys in "SHOW CREATE TABLE" with MySQL. | tidb-test=pr/2312 #51294
Conversation
Welcome @negachov! |
Hi @negachov. Thanks for your PR. I'm waiting for a pingcap member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Hi @negachov. Thanks for your PR. PRs from untrusted users cannot be marked as trusted with I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test all |
@negachov: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
1 similar comment
@negachov: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
You can click the "Details" link of tiprow_fast_test to see the failed tests
|
/retest |
/test unit-test |
@negachov: The specified target(s) for
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #51294 +/- ##
=================================================
- Coverage 72.4334% 55.4490% -16.9844%
=================================================
Files 1481 1593 +112
Lines 365290 628072 +262782
=================================================
+ Hits 264592 348260 +83668
- Misses 81150 256036 +174886
- Partials 19548 23776 +4228
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…fk-without-current-schema
The tests for foreign_key and foreign_key1 are failing, so it seems necessary to revise the test cases.
|
@negachov We will change the tests in private repo soon. Let's start review |
@crazycs520 @hawkingrei Cc: @lance6716 If alternative revisions are preferred, providing guidance on the policy may enable the creation of a diff, so comments would be appreciated. 🙇 |
This looks like a good change to make. Please add a release note to the description and add a link to an issue (create one if needed). |
/assign @ywqzzy |
@negachov could you:
|
…fk-without-current-schema
If there are no existing issues matching, I will create a new issue. |
/test mysql-test |
@dveeden: The specified target(s) for
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/check-issue-triage-complete |
/retest |
1 similar comment
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dveeden, lance6716, zimulala The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
…fk-without-current-schema
@lance6716 plz /retest |
What problem does this PR solve?
Issue Number: ref #51837
Problem Summary:
I'm using parallel_tests with Ruby.
I copy the schema for use, but since foreign keys have schemas, I need to fix the discrepancy in results.
ex)
It's a quick hack, so it's not a good fix.
What changed and how does it work?
Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.