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

parser: support for flush status #7369

Merged
merged 5 commits into from Aug 13, 2018

Conversation

@imtbkcat
Copy link
Contributor

commented Aug 13, 2018

What problem does this PR solve?

Fix #7352 to make some JDBC test cases happy.

What is changed and how it works?

Support flush status syntax in parser. The implementation is a dummy one.

Check List

Tests

  • Unit test

Code changes

  • Has exported function/method change

Side effects
no

Related changes

Copy link
Member

left a comment

LGTM

@imtbkcat

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2018

/run-all-tests

@shenli
shenli approved these changes Aug 13, 2018
Copy link
Member

left a comment

LGTM

@imtbkcat

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2018

/run-all-tests

@tiancaiamao tiancaiamao changed the title parser: Support for flush status parser: support for flush status Aug 13, 2018
imtbkcat added 2 commits Aug 13, 2018
@imtbkcat imtbkcat merged commit 87b16fc into pingcap:master Aug 13, 2018
4 checks passed
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
@imtbkcat imtbkcat deleted the imtbkcat:fix7352 branch Aug 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.