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