[ 
https://issues.apache.org/jira/browse/HADOOP-18920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17776597#comment-17776597
 ] 

ASF GitHub Bot commented on HADOOP-18920:
-----------------------------------------

haiyang1987 commented on code in PR #6146:
URL: https://github.com/apache/hadoop/pull/6146#discussion_r1363627747


##########
hadoop-common-project/hadoop-common/src/main/resources/core-default.xml:
##########
@@ -2512,12 +2512,22 @@ The switch to turn S3A auditing on or off.
 </property>
 
 <property>
-    <name>ipc.server.log.slow.rpc</name>
-    <value>false</value>
-    <description>This setting is useful to troubleshoot performance issues for
-     various services. If this value is set to true then we log requests that
-     fall into 99th percentile as well as increment RpcSlowCalls counter.
-    </description>
+  <name>ipc.server.log.slow.rpc</name>
+  <value>false</value>
+  <description>This setting is useful to troubleshoot performance issues for
+   various services. If this value is set to true then we log requests that
+   fall into 99th percentile as well as increment RpcSlowCalls counter.
+  </description>
+</property>

Review Comment:
   Thanks @ZanderXu  quick review, has been updated PR





> RPC Metrics : Optimize logic for log slow RPCs
> ----------------------------------------------
>
>                 Key: HADOOP-18920
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18920
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Haiyang Hu
>            Assignee: Haiyang Hu
>            Priority: Major
>              Labels: pull-request-available
>
> HADOOP-12325 implement a capability where "slow" RPCs are logged in NN log.
> Current processing logic is the "slow" RPCs are to be those whose processing 
> time is outside 3 standard deviation.
> However, in practice it is found that many logs of slow rpc are currently 
> output, and sometimes RPCs with a processing time of 1ms are also declared as 
> slow, this is not in line with actual expectations.
> Therefore, consider optimize the logic conditions of slow RPC and add a 
> `logSlowRPCThresholdMs` variable to judge whether the current RPCas slow so 
> that the expected slow RPC log can be logger.
> for `logSlowRPCThresholdMs`, we can support dynamic refresh to facilitate 
> adjustments based on the actual operating conditions of the hdfs cluster.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to