-
Notifications
You must be signed in to change notification settings - Fork 105
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
[to #556] slowlog: attach cluster_id and pd_addresses to slow log properties #557
Conversation
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
aaabdd2
Codecov Report
@@ Coverage Diff @@
## master #557 +/- ##
============================================
- Coverage 32.23% 32.05% -0.19%
Complexity 1332 1332
============================================
Files 278 279 +1
Lines 17345 17372 +27
Branches 1975 1980 +5
============================================
- Hits 5591 5568 -23
- Misses 11145 11194 +49
- Partials 609 610 +1
Continue to review full report at Codecov.
|
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-3.1 in PR #559 |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-3.2 in PR #560 |
Signed-off-by: iosmanthus myosmanthustree@gmail.com
What problem does this PR solve?
Issue Number: to #556
Problem Description:
Attach
cluster_id
andpd_addresses
to slow log to distinguish the request destination while handling multiple TiKV clusters.What is changed and how does it work?
Add
withFields
forSlowLogImpl
, and use this interface to attach info dynamically while constructing the slow logger.Code changes
Check List for Tests
This PR has been tested by at least one of the following methods:
Related changes