Skip to content

TwoPhaseCommiter log should be debug level#334

Merged
ti-srebot merged 2 commits into
tikv:masterfrom
humengyu2012:twoPhaseCommiter-log
Nov 20, 2021
Merged

TwoPhaseCommiter log should be debug level#334
ti-srebot merged 2 commits into
tikv:masterfrom
humengyu2012:twoPhaseCommiter-log

Conversation

@humengyu2012

Copy link
Copy Markdown
Contributor

The log of org.tikv.txn.TwoPhaseCommitter is too large, which will cause the really useful logs to be hidden.

image

Signed-off-by: humengyu <hellohumengyu@gmail.com>

@sunxiaoguang sunxiaoguang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@sunxiaoguang

Copy link
Copy Markdown
Member

@marsishandsome Can you double check if this change affects on going tests.

@zz-jason zz-jason left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@sunxiaoguang

Copy link
Copy Markdown
Member

/merge

@ti-srebot

Copy link
Copy Markdown
Collaborator

/run-all-tests

@ti-srebot ti-srebot merged commit 5ae87b1 into tikv:master Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants