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

executor, types: format output #4372

Merged
merged 4 commits into from Aug 30, 2017

Conversation

Projects
None yet
3 participants
@lkk2003rty
Contributor

lkk2003rty commented Aug 29, 2017

fix #4074
fix #4299

lkk2003rty added some commits Aug 29, 2017

@zz-jason zz-jason changed the title from format output to executor, types: format output Aug 29, 2017

@zz-jason

This comment has been minimized.

Show comment
Hide comment
@zz-jason
Member

zz-jason commented Aug 29, 2017

@breeswish PTAL

@shenli

This comment has been minimized.

Show comment
Hide comment
@shenli

shenli Aug 30, 2017

Member

LGTM
@lkk2003rty Thanks!

Member

shenli commented Aug 30, 2017

LGTM
@lkk2003rty Thanks!

@shenli

shenli approved these changes Aug 30, 2017

@shenli shenli merged commit 271f0c0 into pingcap:master Aug 30, 2017

3 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details

dbjoa added a commit to cloud-pi/tidb that referenced this pull request Aug 30, 2017

executor, types: format output (pingcap#4372)
Escape special chars and handle quote marks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment