Skip to content

executor: update the memory usage format in expensive logs (#12809)#16313

Merged
zz-jason merged 3 commits intopingcap:release-3.0from
qw4990:release-3.0-12809
Apr 14, 2020
Merged

executor: update the memory usage format in expensive logs (#12809)#16313
zz-jason merged 3 commits intopingcap:release-3.0from
qw4990:release-3.0-12809

Conversation

@qw4990
Copy link
Contributor

@qw4990 qw4990 commented Apr 13, 2020

cherry-pick for #12809

Copy link
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 13, 2020
@qw4990
Copy link
Contributor Author

qw4990 commented Apr 14, 2020

PTAL @XuHuaiyu @SunRunAway

Copy link
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

@XuHuaiyu XuHuaiyu added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Apr 14, 2020
@qw4990
Copy link
Contributor Author

qw4990 commented Apr 14, 2020

Please help us to merge this PR @zz-jason

@SunRunAway SunRunAway added the status/can-merge Indicates a PR has been approved by a committer. label Apr 14, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 14, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Apr 14, 2020

@qw4990 merge failed.

@zz-jason
Copy link
Member

/run-common-test

@zz-jason zz-jason removed the request for review from SunRunAway April 14, 2020 08:39
@zz-jason zz-jason merged commit 3c96a15 into pingcap:release-3.0 Apr 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/execution SIG execution status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. type/usability type/3.0-cherry-pick

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants