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

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

wzk784533 opened a new pull request, #6666:
URL: https://github.com/apache/hadoop/pull/6666

   Description of PR
   https://issues.apache.org/jira/browse/HADOOP-19111
   
   Removing redundant debug message about client info
   
   when sending rpc request,the debug message print client name(which is 
getName()) twice,this is confusing.
   
   like:
   
   2024-03-16 11:58:49,564 DEBUG ipc.Client: :1113 IPC Client (382750013) 
connection to kdcserver/172.20.10.12:8888 from 
nn/[[email protected]](mailto:[email protected])IPC Client (382750013) connection 
to kdcserver/172.20.10.12:8888 from 
nn/[[email protected]](mailto:[email protected]) sending #0 
org.apache.hadoop.hdfs.protocol.ClientProtocol.getListing
   
   the debug message above should delete the bold font part
   (can't set strikethrough like jira,so use bold fond)
   
   ### How was this patch tested?
   The code chaning is easy to understand. And I have it tested in Junit with 
other cases in local
   
   ### For code changes:
   
                   LOG.debug(getName() + "{} sending #{} {}", getName(), 
call.id, call.rpcRequest);
   
   abut the code above,there are two reasons to change:
   a: getName() + and the {}placeholder are duplicate
   b: To avoid String copy, we should use placehold rather than String +




> ipc client print client info message duplicate
> ----------------------------------------------
>
>                 Key: HADOOP-19111
>                 URL: https://issues.apache.org/jira/browse/HADOOP-19111
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: common
>            Reporter: Zhongkun Wu
>            Priority: Trivial
>              Labels: pull-request-available
>
> when sending rpc request,the debug message print clent name twice,this is 
> confusing.
> like:
>  
> 2024-03-16 11:58:49,564 DEBUG ipc.Client: :1113 {-}{color:#4c9aff}IPC Client 
> (382750013) connection to kdcserver/172.20.10.12:8888 from 
> nn/[email protected]{color}{-}IPC Client (382750013) connection to 
> kdcserver/172.20.10.12:8888 from nn/[email protected] sending #0 
> org.apache.hadoop.hdfs.protocol.ClientProtocol.getListing
>  
>  
> the debug message above should delete the blue infomation with strikethrough



--
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