Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plan, executor: check b.err when building logical plan(#7114)(#7204) #7225

Merged
merged 5 commits into from Aug 2, 2018

Conversation

@XuHuaiyu
Copy link
Contributor

commented Aug 1, 2018

cherry-pick #7144 #7204

@XuHuaiyu

This comment has been minimized.

Copy link
Contributor Author

commented Aug 1, 2018

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

@winkyao
Copy link
Member

left a comment

LGTM

@shenli
Copy link
Member

left a comment

LGTM

@XuHuaiyu XuHuaiyu dismissed stale reviews from shenli and winkyao via 4ee2dcb Aug 1, 2018

@XuHuaiyu

This comment has been minimized.

Copy link
Contributor Author

commented Aug 1, 2018

/run-unit-test tidb-test=release-2.0 tikv=release-2.0 pd=release-2.0

@winoros
winoros approved these changes Aug 1, 2018
@zz-jason

This comment has been minimized.

Copy link
Member

commented Aug 2, 2018

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

@zz-jason
Copy link
Member

left a comment

LGTM

@zz-jason zz-jason added the status/LGT2 label Aug 2, 2018

@zz-jason zz-jason merged commit 64673ee into pingcap:release-2.0 Aug 2, 2018

4 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci-tidb/build Jenkins job succeeded.
Details
license/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.