hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arpit Agarwal (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-6578) add toString method to DatanodeStorage etc for easier debugging
Date Fri, 20 Jun 2014 18:53:26 GMT

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

Arpit Agarwal commented on HDFS-6578:
-------------------------------------

bq. It seems to be nice to add a toString() method for DatanodeStorage class, so we can print
out its key info easier while doing debuging.
The StorageID uniquely identifies the storage, but if you want to print out the state and
type that should be fine.

> add toString method to DatanodeStorage etc for easier debugging
> ---------------------------------------------------------------
>
>                 Key: HDFS-6578
>                 URL: https://issues.apache.org/jira/browse/HDFS-6578
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Yongjun Zhang
>            Assignee: Yongjun Zhang
>
> It seems to be nice to add a toString() method for DatanodeStorage class, so we can print
out its key info easier while doing debuging.
> Another thing is, in the end of BlockManager#processReport, there is the following message,
> {code}
>    blockLog.info("BLOCK* processReport: from storage " + storage.getStorageID()
>         + " node " + nodeID + ", blocks: " + newReport.getNumberOfBlocks()
>         + ", processing time: " + (endTime - startTime) + " msecs");
>     return !node.hasStaleStorages();
> {code}
> We could add node.hasStaleStorages() to the log, and possibly replace storage.getSorateID()
with the suggested storage.toString().
> Any comments? thanks.



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

Mime
View raw message