Skip to content

expression: let testEvaluatorSuite run serially to avoid affect other unit tests since it uses failpoint (#11310)#11366

Merged
zz-jason merged 2 commits into
pingcap:release-3.0from
qw4990:release-3.0-pb-panic
Jul 23, 2019
Merged

expression: let testEvaluatorSuite run serially to avoid affect other unit tests since it uses failpoint (#11310)#11366
zz-jason merged 2 commits into
pingcap:release-3.0from
qw4990:release-3.0-pb-panic

Conversation

@qw4990
Copy link
Copy Markdown
Contributor

@qw4990 qw4990 commented Jul 22, 2019

cherry-pick for #11310.

@qw4990 qw4990 requested review from XuHuaiyu and zz-jason July 22, 2019 07:45
@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

/run-all-tests tikv=release-3.0 pd=release-3.0

@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

/run-check_dev_2 tikv=release-3.0 pd=release-3.0

@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

/run-all-tests tikv=release-3.0 pd=release-3.0

1 similar comment
@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

/run-all-tests tikv=release-3.0 pd=release-3.0

@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

/run-all-tests

@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

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

@qw4990
Copy link
Copy Markdown
Contributor Author

qw4990 commented Jul 22, 2019

/rebuild

Copy link
Copy Markdown
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 the status/LGT1 Indicates that a PR has LGTM 1. label Jul 22, 2019
Copy link
Copy Markdown
Contributor

@XuHuaiyu XuHuaiyu 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 23, 2019
@zz-jason zz-jason merged commit 91ae309 into pingcap:release-3.0 Jul 23, 2019
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. type/bugfix This PR fixes a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants