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

Xiaoyu Yao commented on HDFS-11492:
-----------------------------------

Thanks [~anu] for working on this. The latest patch looks pretty good to me. 
Just have some minor questions.

*ContainerMangerImpl.java*
Line 622-632: NIT: a good candidate to use JDK8 stream/filter/map?

*DatanodeStateMachine.java*
Line 276: NIT: typo in commandProcessThead

Line 280: should we limit the maximum restart attempts? 

> Ozone: Add the ability to handle sendContainerReport Command
> ------------------------------------------------------------
>
>                 Key: HDFS-11492
>                 URL: https://issues.apache.org/jira/browse/HDFS-11492
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>    Affects Versions: HDFS-7240
>            Reporter: Anu Engineer
>            Assignee: Anu Engineer
>         Attachments: HDFS-11492-HDFS-7240.001.patch
>
>
> Once a container report is ready datanodes sends that information to SCM. SCM 
> returns a command asking the data node to send container report. Add the 
> ability to handle this command on datanode side and send the actual container 
> Report.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
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