-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
VReplication: Add stream DDL processing stats #15769
Conversation
Signed-off-by: Matt Lord <[email protected]>
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #15769 +/- ##
==========================================
+ Coverage 68.39% 68.45% +0.06%
==========================================
Files 1556 1558 +2
Lines 195422 195836 +414
==========================================
+ Hits 133666 134069 +403
- Misses 61756 61767 +11 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
bps.TableCopyTimings = stats.NewTimings("", "", "Table") | ||
bps.PartialQueryCacheSize = stats.NewCountersWithMultiLabels("", "", []string{"type"}) | ||
bps.PartialQueryCount = stats.NewCountersWithMultiLabels("", "", []string{"type"}) | ||
bps.ThrottledCounts = stats.NewCountersWithMultiLabels("", "", []string{"throttler", "component"}) | ||
bps.DDLEventActions = stats.NewCountersWithSingleLabel("", "", "action") |
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.
Any idea whether these tags are case-sensitive? I mean when filtering using PromQL. In this case, there is only 1 tag, but for multi-label counters, you might want to filter that way.
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.
Yes, I believe they are. For example this returns values in grafana with prometheus:
vttablet_errors{error_code="ABORTED"}
But these do not:
vttablet_errors{error_code="aborted"}
vttablet_errors{error_CODE="ABORTED"}
* Document new stat added in vitessio/vitess#15769 Signed-off-by: Matt Lord <[email protected]> * Minor touchup Signed-off-by: Matt Lord <[email protected]> --------- Signed-off-by: Matt Lord <[email protected]>
Description
This adds stats to track the DDL processing actions taken in currently running VReplication streams based on the workflow's configured value of
--on-ddl
.Manual test
Final output:
Related Issue(s)
Checklist