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

server, session: refine mysql handshake error log #15731

Merged
merged 2 commits into from
Mar 28, 2020
Merged

server, session: refine mysql handshake error log #15731

merged 2 commits into from
Mar 28, 2020

Conversation

lysu
Copy link
Contributor

@lysu lysu commented Mar 26, 2020

What problem does this PR solve?

Problem Summary:

now, we log out too many EOF log due to port health check

What is changed and how it works?

make handshake EOF error be logged as INFO level(or debug?) and suspend stack info for them

What's Changed:

How it Works:

log inside handshake method

Related changes

  • Need to cherry-pick to the release branch

Check List

Tests

  • Manual test

Side effects

  • n/a

Release note

refine mysql mysql handshake error log


This change is Reviewable

@codecov
Copy link

codecov bot commented Mar 26, 2020

Codecov Report

Merging #15731 into master will decrease coverage by 0.1225%.
The diff coverage is 26.923%.

@@               Coverage Diff                @@
##             master     #15731        +/-   ##
================================================
- Coverage   80.5057%   80.3831%   -0.1226%     
================================================
  Files           504        502         -2     
  Lines        134742     134369       -373     
================================================
- Hits         108475     108010       -465     
- Misses        17803      17856        +53     
- Partials       8464       8503        +39

@tiancaiamao
Copy link
Contributor

LGTM

@tiancaiamao tiancaiamao added the status/LGT1 Indicates that a PR has LGTM 1. label Mar 28, 2020
Copy link
Member

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

jackysp commented Mar 28, 2020

/merge

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

sre-bot commented Mar 28, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Mar 28, 2020

cherry pick to release-3.0 in PR #15798

sre-bot pushed a commit to sre-bot/tidb that referenced this pull request Mar 28, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot
Copy link
Contributor

sre-bot commented Mar 28, 2020

cherry pick to release-3.1 in PR #15799

@sre-bot
Copy link
Contributor

sre-bot commented Mar 28, 2020

cherry pick to release-4.0 in PR #15800

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/server status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. type/usability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants