[ 
https://issues.apache.org/jira/browse/HDFS-11492?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anu Engineer updated HDFS-11492:
--------------------------------
    Attachment: HDFS-11492-HDFS-7240.003.patch

Thank you for your comments [~xyao]. The new patch addresses all comments 
except one.

bq. Line 622-632: NIT: a good candidate to use JDK8 stream/filter/map?
fixed.
bq. Line 276: NIT: typo in commandProcessThead
fixed.
bq. Line 280: should we limit the maximum restart attempts?
I am very reluctant to do this, because we have no safe way of only disabling 
this thread. That is if statemachine is running, we must have command processor 
also running. The hope is that if we get errors in some handler, that handler 
will handle it correctly and set the shutdown flag for errors which are not 
recoverable. if the unhandledException handler is getting run, that is probably 
a Bug. We can add a counter to  monitor this condition in  data nodes via JMX 
later.

> 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, 
> HDFS-11492-HDFS-7240.002.patch, HDFS-11492-HDFS-7240.003.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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to