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

Rakesh R commented on HDFS-4266:
--------------------------------

Thanks [~ikelly] for the review.

bq.The patch makes an ack quorum mandatory. This breaks existing configs. The 
ack quorum should default to the write quorum, if the configuration is missing.
Hdfs configuration will return the 'quorumSize' if the configuration is missing.
{code}
ackQuorumSize = conf.getInt(BKJM_BOOKKEEPER_ACK_QUORUM_SIZE, quorumSize);
{code}

I've included test case to verify this behavior. Could you have a look at the 
latest patch.Thanks!

> BKJM: Separate write and ack quorum
> -----------------------------------
>
>                 Key: HDFS-4266
>                 URL: https://issues.apache.org/jira/browse/HDFS-4266
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ha
>            Reporter: Ivan Kelly
>            Assignee: Rakesh R
>         Attachments: 001-HDFS-4266.patch, 002-HDFS-4266.patch
>
>
> BOOKKEEPER-208 allows the ack and write quorums to be different sizes to 
> allow writes to be unaffected by any bookie failure. BKJM should be able to 
> take advantage of this.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to