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

Weiwei Yang commented on HDFS-12329:
------------------------------------

Hi [~msingh]

The patch makes sense to me, I have 2 questions and if I can get them clear I 
will +1 on this patch,

1. In MiniOzoneCluster the patch added a method {{waitDatanodeOzoneReady}} to 
check if a datanode is in RUNNING state. I am wondering why this cannot be done 
via {{waitOzoneReady}}. Because SCMNodeManager should manage the states of all 
DNs, if they are rebooted, SCM should be aware and keep number of healthy nodes 
updated.

2. As you mentioned in HDFS-12354 this patch fixed the UT failure in TestKeys, 
I am wondering what was the problem and how that is fixed in this patch.

Thanks

> Ozone: Ratis: Readonly calls in XceiverClientRatis should use sendReadOnly
> --------------------------------------------------------------------------
>
>                 Key: HDFS-12329
>                 URL: https://issues.apache.org/jira/browse/HDFS-12329
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>    Affects Versions: HDFS-7240
>            Reporter: Mukul Kumar Singh
>            Assignee: Mukul Kumar Singh
>             Fix For: HDFS-7240
>
>         Attachments: HDFS-12329-HDFS-7240.001.patch, 
> HDFS-12329-HDFS-7240.002.patch
>
>
> Currently both write and readonly calls in Ratis use RaftClient.send which 
> enqueues the the request to the raft log and is processed later when the log 
> entry is consumed.
> Readonly call can be optimized by using RaftClient.sendReadOnly which will 
> directly query the RaftServer for a particular request.
> This jira will be used to discuss this issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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

Reply via email to