Skip to content

Conversation

zimulala
Copy link
Contributor

cherry-pick #8242.

@zz-jason
Copy link
Member

LGTM

@zz-jason zz-jason added the status/LGT1 Indicates that a PR has LGTM 1. label Nov 12, 2018
Copy link
Member

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

@jackysp
Copy link
Member

jackysp commented Nov 12, 2018

/run-all-tests pd=release-2.1 tikv=release-2.1 tidb-test=release-2.1

@jackysp jackysp added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Nov 12, 2018
@winkyao winkyao added the priority/release-blocker This issue blocks a release. Please solve it ASAP. label Nov 12, 2018
@zimulala
Copy link
Contributor Author

/run-all-tests pd=release-2.1 tikv=release-2.1 tidb-test=release-2.1

@zz-jason zz-jason merged commit 43049f4 into pingcap:release-2.1 Nov 12, 2018
@zimulala zimulala deleted the cp-8242 branch November 12, 2018 07:49
zimulala added a commit to zimulala/tidb that referenced this pull request Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/release-blocker This issue blocks a release. Please solve it ASAP. status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants