chenBright commented on code in PR #3549:
URL: https://github.com/apache/brpc/pull/3549#discussion_r4033525568
##########
src/brpc/builtin/prometheus_metrics_service.cpp:
##########
@@ -181,6 +181,12 @@ bool PrometheusMetricsDumper::DumpLatencyRecorderSuffix(
if (!si->IsComplete()) {
return true;
}
+ // The average latency can not be a quantile series of the summary below,
+ // because the quantile label must be parsable as a float. Dump it as a
+ // separate gauge, which is the same as the multi dimension one does.
+ *_os << "# HELP " << si->metric_name << "_avg_latency" << '\n'
+ << "# TYPE " << si->metric_name << "_avg_latency gauge\n"
+ << si->metric_name << "_avg_latency " << si->latency_avg << '\n';
Review Comment:
The PR description has been updated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
