duongnguyen0 commented on code in PR #3637:
URL: https://github.com/apache/ozone/pull/3637#discussion_r933406309
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/impl/HddsDispatcher.java:
##########
@@ -304,10 +313,25 @@ private ContainerCommandResponseProto dispatchRequest(
audit(action, eventType, params, AuditEventStatus.FAILURE, ex);
return ContainerUtils.logAndReturnError(LOG, ex, msg);
}
+ long handlingStartTime = Time.monotonicNow();
responseProto = handler.handle(msg, container, dispatcherContext);
if (responseProto != null) {
+ long requestDurationMs = Time.monotonicNow() - startTime;
+ long handlingDurationMs = Time.monotonicNow() - handlingStartTime;
Review Comment:
+1.
If we expose those timings as metrics with proper quantile, maybe only
`avg` and `p99`, we may not need warning logs at all. p99 would report the
outliners which we can proactively look at or create alerts on.
--
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]
