[
https://issues.apache.org/jira/browse/HDDS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16464244#comment-16464244
]
Anu Engineer commented on HDDS-17:
----------------------------------
[~elek] and [~nandakumar131] Thanks for the comments. I have uploaded patch v2
that takes care of all the comments.
bq. Instead of Long we can use ContainerID
Fixed, Good catch.
bq. In Node2ContainerMap#updateDatanodeMap can we throw exception if the node
is not already present.
Fixed.
Some more changes in this version of the patch.
# Made the class thread safe.
# Added a function to get the container list given a datanode.
> Add node to container map class to simplify state in SCM
> --------------------------------------------------------
>
> Key: HDDS-17
> URL: https://issues.apache.org/jira/browse/HDDS-17
> Project: Hadoop Distributed Data Store
> Issue Type: Improvement
> Components: SCM
> Reporter: Anu Engineer
> Assignee: Anu Engineer
> Priority: Major
> Fix For: 0.2.1
>
> Attachments: HDDS-17.001.patch, HDDS-17.002.patch
>
>
> Current SCM state map is maintained in nodeStateManager. This first of
> several refactoring to make it independent and small classes.
--
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
