Skip to content

Conversation

zimulala
Copy link
Contributor

@zimulala zimulala commented Jul 5, 2018

What have you changed? (mandatory)

Get the correct result of "show create table" when executing "add index".
Fix #6991

What are the type of the changes (mandatory)?

  • Improvement

How has this PR been tested (mandatory)?

unit test

@zimulala zimulala added the sig/execution SIG execution label Jul 5, 2018
@zimulala
Copy link
Contributor Author

zimulala commented Jul 5, 2018

/run-all-tests

@shenli
Copy link
Member

shenli commented Jul 5, 2018

LGTM

@shenli shenli added type/bugfix This PR fixes a bug. status/LGT1 Indicates that a PR has LGTM 1. labels Jul 5, 2018
@zimulala
Copy link
Contributor Author

zimulala commented Jul 5, 2018

/run-all-tests

Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zz-jason zz-jason added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Jul 5, 2018
@zz-jason zz-jason merged commit 48c0fcd into pingcap:master Jul 5, 2018
@zimulala zimulala deleted the adjust-show-create branch July 5, 2018 07:16
zimulala added a commit to zimulala/tidb that referenced this pull request Aug 2, 2018
coocood pushed a commit that referenced this pull request Aug 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/execution SIG execution status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants