[ 
https://issues.apache.org/jira/browse/HDDS-567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16636149#comment-16636149
 ] 

Anu Engineer commented on HDDS-567:
-----------------------------------

You can fix the below issue while committing. +1. No need for another patch.
 * {{ContainerStateManager.java:Line 218:}}
  {{Event and State Transition Mapping:}} I think we should leave this comment 
as is. In the current patch we have replaced it as {{Event and State Transition 
*ContainerManager*}}

Really appreciate you taking care of this, this was long over due and 
appreciate you fixing this.

I am *not* committing since the trunk is *broken* right now. Please feel free 
to commit when the trunk is healthy.

> Rename Mapping to ContainerManager in SCM
> -----------------------------------------
>
>                 Key: HDDS-567
>                 URL: https://issues.apache.org/jira/browse/HDDS-567
>             Project: Hadoop Distributed Data Store
>          Issue Type: Improvement
>          Components: SCM
>            Reporter: Nanda kumar
>            Assignee: Nanda kumar
>            Priority: Major
>         Attachments: HDDS-567.000.patch
>
>
> In SCM we have an interface named {{Mapping}} which is actually for container 
> management, it is better to rename this interface as {{ContainerManager}} 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to