[
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: [email protected]
For additional commands, e-mail: [email protected]
