-
Notifications
You must be signed in to change notification settings - Fork 283
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
mysql-sink: add more performance related metrics #702
Conversation
/run-integration-tests |
/run-integration-tests |
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.
Rest LGTM
bucketSizeCounter = prometheus.NewCounterVec( | ||
prometheus.CounterOpts{ | ||
Namespace: "ticdc", | ||
Subsystem: "sink", | ||
Name: "bucket_size", | ||
Help: "size of the DML bucket", | ||
}, []string{"capture", "changefeed", "bucket"}) |
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.
Looks like gauge is more suitable for size.
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.
Because the sink worker flushes DMLs every 100ms, it is hard to track real bucket size if using gauge, so here use a counter instead.
/run-integration-tests |
Codecov Report
@@ Coverage Diff @@
## master #702 +/- ##
===========================================
Coverage 31.4172% 31.4172%
===========================================
Files 90 90
Lines 9307 9307
===========================================
Hits 2924 2924
Misses 6143 6143
Partials 240 240 |
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
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
/merge |
/run-integration-tests |
/merge |
What problem does this PR solve?
Solve #685
What is changed and how it works?
Check List
Tests
Related changes
Release note