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

ASF GitHub Bot commented on HDFS-17431:
---------------------------------------

haiyang1987 commented on PR #6643:
URL: https://github.com/apache/hadoop/pull/6643#issuecomment-2008603273

   Thanks @wzk784533 @ayushtkn @dineshchitlangia for your review.
   
   I found that the log format  showed some problems, such as the mentioned in 
this issue https://github.com/apache/hadoop/pull/6635. 
   ```
   
   2024-03-13 23:07:05,401 WARN  datanode.DataNode 
(BlockRecoveryWorker.java:run(623)) 
[org.apache.hadoop.hdfs.server.datanode.BlockRecoveryWorker$1@54e291ac] -
   recover Block: RecoveringBlock{BP-xxx:blk_xxx_xxx; getBlockSize()=0; 
corrupt=false; offset=-1; locs=[DatanodeInfoWithStorage[dn1:50010,null,null], 
DatanodeInfoWithStorage[dn2:50010,null,null]]; cachedLocs=[]}
   FAILED: {} org.apache.hadoop.ipc.RemoteException(java.io.IOException): The 
recovery id 28577373754 does not match current recovery id 28578772548 for 
block BP-xxx:blk_xxx_xxx
           at 
org.apache.hadoop.hdfs.server.namenode.FSNamesystem.commitBlockSynchronization(FSNamesystem.java:4129)
           at 
org.apache.hadoop.hdfs.server.namenode.NameNodeRpcServer.commitBlockSynchronization(NameNodeRpcServer.java:1184)
           at 
org.apache.hadoop.hdfs.protocolPB.DatanodeProtocolServerSideTranslatorPB.commitBlockSynchronization(DatanodeProtocolServerSideTranslatorPB.java:310)
           at 
org.apache.hadoop.hdfs.protocol.proto.DatanodeProtocolProtos$DatanodeProtocolService$2.callBlockingMethod(DatanodeProtocolProtos.java:34391)
           at 
org.apache.hadoop.ipc.ProtobufRpcEngine2$Server$ProtoBufRpcInvoker.call(ProtobufRpcEngine2.java:635)
           at 
org.apache.hadoop.ipc.ProtobufRpcEngine2$Server$ProtoBufRpcInvoker.call(ProtobufRpcEngine2.java:603)
           at 
org.apache.hadoop.ipc.ProtobufRpcEngine2$Server$ProtoBufRpcInvoker.call(ProtobufRpcEngine2.java:587)
           at org.apache.hadoop.ipc.RPC$Server.call(RPC.java:1137)
           at org.apache.hadoop.ipc.Server$RpcCall.run(Server.java:1236)
           at org.apache.hadoop.ipc.Server$RpcCall.run(Server.java:1134)
           at java.security.AccessController.doPrivileged(Native Method)
           at javax.security.auth.Subject.doAs(Subject.java:422)
           at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:2005)
           at org.apache.hadoop.ipc.Server$Handler.run(Server.java:3360)
   
           at org.apache.hadoop.ipc.Client.getRpcResponse(Client.java:1579)
           at org.apache.hadoop.ipc.Client.call(Client.java:1511)
           at org.apache.hadoop.ipc.Client.call(Client.java:1402)
           at 
org.apache.hadoop.ipc.ProtobufRpcEngine2$Invoker.invoke(ProtobufRpcEngine2.java:268)
           at 
org.apache.hadoop.ipc.ProtobufRpcEngine2$Invoker.invoke(ProtobufRpcEngine2.java:142)
           at com.sun.proxy.$Proxy17.commitBlockSynchronization(Unknown Source)
           at 
org.apache.hadoop.hdfs.protocolPB.DatanodeProtocolClientSideTranslatorPB.commitBlockSynchronization(DatanodeProtocolClientSideTranslatorPB.java:342)
           at 
org.apache.hadoop.hdfs.server.datanode.BlockRecoveryWorker$RecoveryTaskContiguous.syncBlock(BlockRecoveryWorker.java:334)
           at 
org.apache.hadoop.hdfs.server.datanode.BlockRecoveryWorker$RecoveryTaskContiguous.recover(BlockRecoveryWorker.java:189)
           at 
org.apache.hadoop.hdfs.server.datanode.BlockRecoveryWorker$1.run(BlockRecoveryWorker.java:620)
           at java.lang.Thread.run(Thread.java:748)
   ```
   
   `LOG.warn("recover Block: {} FAILED: {}", b, e);`
    it invoke e will print the entire trace. so for the second placeholders is 
meaningless,  i think choose to remove for the second placeholders or  change e 
to e.toString()
   
   Hi sir @ayushtkn @dineshchitlangia @wzk784533 what dou you think?
   Thanks~
   
   




> Fix log format for BlockRecoveryWorker#recoverBlocks
> ----------------------------------------------------
>
>                 Key: HDFS-17431
>                 URL: https://issues.apache.org/jira/browse/HDFS-17431
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Haiyang Hu
>            Assignee: Haiyang Hu
>            Priority: Major
>              Labels: pull-request-available
>
> Fix log format for BlockRecoveryWorker#recoverBlocks



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