[ 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