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

Tsz Wo Nicholas Sze commented on HDFS-6794:
-------------------------------------------

For the findbugs warnings, we should keep using dn instead of node in the code 
below.
{code}
     if (node == null) {
-      throw new IOException("Cannot mark " + b
-          + " as corrupt because datanode " + dn + " (" + dn.getDatanodeUuid()
+      throw new IOException("Cannot mark " + blk
+          + " as corrupt because datanode " + node + " (" + 
node.getDatanodeUuid()
           + ") does not exist");
     }
{code}

> Update BlockManager methods to use DatanodeStorageInfo where possible
> ---------------------------------------------------------------------
>
>                 Key: HDFS-6794
>                 URL: https://issues.apache.org/jira/browse/HDFS-6794
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Arpit Agarwal
>            Assignee: Arpit Agarwal
>         Attachments: HDFS-6794.01.patch, HDFS-6794.02.patch, 
> HDFS-6794.03.patch
>
>
> Post HDFS-2832, BlockManager methods can be updated to accept 
> DatanodeStorageInfo instead of (DatanodeDescriptor + StorageID).



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to