haiyang1987 commented on code in PR #6146:
URL: https://github.com/apache/hadoop/pull/6146#discussion_r1349497618
##########
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java:
##########
@@ -568,12 +579,15 @@ public long getPurgeIntervalNanos() {
* @param methodName - RPC Request method name
* @param details - Processing Detail.
*
- * if this request took too much time relative to other requests
- * we consider that as a slow RPC. 3 is a magic number that comes
- * from 3 sigma deviation. A very simple explanation can be found
- * by searching for 68-95-99.7 rule. We flag an RPC as slow RPC
- * if and only if it falls above 99.7% of requests. We start this logic
- * only once we have enough sample size.
+ * if this request took too much time relative to other requests,
+ * and it took time exceed `logSlowRPCThresholdMs` we consider that as a
slow RPC.
Review Comment:
Thanks @xinglin for your suggestion, i will update it later.
##########
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java:
##########
@@ -587,10 +601,10 @@ void logSlowRpcCalls(String methodName, Call call,
final double threeSigma = rpcMetrics.getProcessingMean() +
(rpcMetrics.getProcessingStdDev() * deviation);
- long processingTime =
- details.get(Timing.PROCESSING, rpcMetrics.getMetricsTimeUnit());
+ long processingTime = details.get(Timing.PROCESSING,
rpcMetrics.getMetricsTimeUnit());
Review Comment:
Get it.
--
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]
