[ 
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

Reply via email to