Skip to content

Conversation

alivxxx
Copy link
Contributor

@alivxxx alivxxx commented Dec 22, 2017

Cherry pick #4801.
PTAL @zimulala

Copy link
Member

@winoros winoros left a comment

Choose a reason for hiding this comment

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

LGTM

@winoros winoros added the status/LGT1 Indicates that a PR has LGTM 1. label Dec 22, 2017
Copy link
Contributor

@zimulala zimulala left a comment

Choose a reason for hiding this comment

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

LGTM

@zimulala
Copy link
Contributor

/run-all-tests tidb-test=release-1.0

@zimulala zimulala added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Dec 22, 2017
@coocood
Copy link
Member

coocood commented Dec 22, 2017

/run-common-tests tidb-test=release-1.0 tidb-private-test=release-1.0

@winoros winoros merged commit f4fb401 into pingcap:release-1.0 Dec 22, 2017
@alivxxx alivxxx deleted the schema branch December 22, 2017 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants