Skip to content

Conversation

winoros
Copy link
Member

@winoros winoros commented Jul 3, 2018

What have you changed? (mandatory)

Cherry pick #6925

What are the type of the changes (mandatory)?

The currently defined types are listed below, please pick one of the types for this PR by removing the others:

Cherry-pick

How has this PR been tested (mandatory)?

new integration test.

@shenli
Copy link
Member

shenli commented Jul 3, 2018

LGTM

@shenli shenli added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 3, 2018
@winoros
Copy link
Member Author

winoros commented Jul 3, 2018

/run-all-tests tikv=release-2.0 pd=release-2.0 tidb-test=pr/561

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 3, 2018
@shenli
Copy link
Member

shenli commented Jul 3, 2018

/run-all-tests tikv=release-2.0 pd=release-2.0 tidb-test=pr/561

@zz-jason zz-jason merged commit bfcea6d into pingcap:release-2.0 Jul 3, 2018
@winoros winoros deleted the pick branch July 4, 2018 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/planner SIG: Planner status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants