Skip to content

executor: replace logger with zap logger (#9521)#10043

Merged
jackysp merged 2 commits intopingcap:release-2.1from
alivxxx:log
Apr 4, 2019
Merged

executor: replace logger with zap logger (#9521)#10043
jackysp merged 2 commits intopingcap:release-2.1from
alivxxx:log

Conversation

@alivxxx
Copy link
Copy Markdown
Contributor

@alivxxx alivxxx commented Apr 4, 2019

Cherry pick #9521

@alivxxx alivxxx added type/enhancement The issue or PR belongs to an enhancement. sig/execution SIG execution type/2.1 cherry-pick labels Apr 4, 2019
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 Apr 4, 2019
@alivxxx
Copy link
Copy Markdown
Contributor Author

alivxxx commented Apr 4, 2019

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

Copy link
Copy Markdown
Contributor

@jackysp jackysp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jackysp jackysp merged commit e87c6f1 into pingcap:release-2.1 Apr 4, 2019
@alivxxx alivxxx deleted the log branch April 4, 2019 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/execution SIG execution status/LGT1 Indicates that a PR has LGTM 1. type/enhancement The issue or PR belongs to an enhancement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants