-
Notifications
You must be signed in to change notification settings - Fork 417
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
feat: support cpu profile api #3279
Conversation
Signed-off-by: Song Gao <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3279 +/- ##
==========================================
- Coverage 70.53% 70.51% -0.01%
==========================================
Files 400 400
Lines 45064 45086 +22
==========================================
+ Hits 31782 31791 +9
- Misses 10742 10755 +13
Partials 2540 2540 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Song Gao <[email protected]>
Signed-off-by: Song Gao <[email protected]>
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.
Markdown check fails
Signed-off-by: Song Gao <[email protected]>
@@ -80,6 +80,10 @@ After version 2.0.0, we added connection-related metrics for source/sink. | |||
|
|||
The numeric types of these metrics can all be monitored using Prometheus. In the next section we will describe how to configure the Prometheus service in eKuiper. | |||
|
|||
View CPU running metrics for a rule | |||
|
|||
- kuiper_rule_cpuUsage: The CPU running indicator of the rule represents the CPU time used by the CPU in the past 30 seconds, in ms. |
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.
Is it better to comply with the metrics naming pattern (https://prometheus.io/docs/practices/naming/): kuiper_rule_cpu_ms?
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.
Doc is not updated
Signed-off-by: Song Gao <[email protected]>
Signed-off-by: Song Gao <[email protected]>
No description provided.