[GitHub] ivankelly commented on issue #1380: addBookieAndCheckCovered in RRQuorumCoverageSet doesn't work correctly if AcqQuorumSize is greater than (WriteQuorumSize+1)/2

[GitBox <gi...@apache.org>]

ivankelly commented on issue #1380: addBookieAndCheckCovered in RRQuorumCoverageSet doesn't work correctly if AcqQuorumSize is greater than (WriteQuorumSize+1)/2
URL: https://github.com/apache/bookkeeper/issues/1380#issuecomment-386551005
 
 
   @reddycharan @sijie 
   
   Yes, it's as @sijie says. checkCovered is not for the write path as such. Not really for the read path either. It's for fencing.
   
   The intuition behind it is, if a entry X has been acknowledged to the client, it has to have been written to a full ack quorum of bookies. Conversely, if at least one bookie in each possible ack quorum has acknowledged the a fence operation, then it is impossible for any entry to be acknowledged to the client from that point on.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services