[ https://issues.apache.org/jira/browse/HDDS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16463837#comment-16463837 ]
Elek, Marton commented on HDDS-17: ---------------------------------- Thanks @Anu, he patch itself looks good to me (except the minor whitespace issue). ((This is just the simple data class to store the information but the integration point (how it will be used) is not visible yet. For example currently there is just one ReportResult with all the expected state but they also could be in different event classes. It depends on the usage. But it's good and safe to commit it now and will see later if the usage requires changes. I guess you have more clean vision about it...)) > 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 > > > 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