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

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

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


##########
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java:
##########
@@ -2509,6 +2518,43 @@ String reconfigureIPCBackoffEnabled(String newVal) {
     return Boolean.toString(clientBackoffEnabled);
   }
 
+  String reconfigureLogSlowRPC(String property, String newVal) throws 
ReconfigurationException {
+    String result = null;
+    try {
+      if (property.equals(IPC_SERVER_LOG_SLOW_RPC)) {

Review Comment:
   I assume this is needed. But why did we NOT have this code for 
`IPC_SERVER_LOG_SLOW_RPC` before?





> 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