[ https://issues.apache.org/jira/browse/HDDS-38?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16471700#comment-16471700 ]
Elek, Marton commented on HDDS-38: ---------------------------------- Thanks [~shashikant] to create this patch and especially to publish the information over jmx. >From the UI point of view we need some operation to get the capacity but >ideally it should be one bug list instead of many small request. It's not >clear for me how this SCMNodeStorageStateMap will be integrated in scm. One >option is to extend a higher level node list call (don't know if we have it >yet or not) with the capacity information. The other option is to extend >SCMNodeStorageStateMxBean with a list operation and do the join on the client >side based on the datanode ids. Some other minor comments: * Some constants are missing from the patch (Or I applied the patch badly). Such as DUPLICATE_DATANODE. * I would use the name SCMNodeStorageStatMXBean (Upper case MX). Just to be consistent with the previous class names. > Add SCMNodeStorage map in SCM class to store storage statistics per Datanode > ---------------------------------------------------------------------------- > > Key: HDDS-38 > URL: https://issues.apache.org/jira/browse/HDDS-38 > Project: Hadoop Distributed Data Store > Issue Type: Bug > Components: SCM > Reporter: Shashikant Banerjee > Assignee: Shashikant Banerjee > Priority: Major > Fix For: 0.2.1 > > Attachments: HDDS-38.00.patch > > > Currently , the storage stats per Datanode are maintained inside > scmNodeManager. This will > move the scmNodeStats for storage outside SCMNodeManager to simplify > refactoring. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org