reddycharan opened a new 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 **QUESTION** Ensemble Size - 9 WriteQuorum Size - 7 AcqQuorum Size - 5 lets assume the following scenario BK - means Bookie BK0 (NonValidResponse) - BK1 (ValidResponse) -BK2 (NonValid) - BK3 (Valid) -BK4 (NonValid) - BK5 (Valid) -BK6 (NonValid) - BK7 (Valid) - BK8 (NonValid) basically False (0) - True (1) - False (2) - True (3) - False (4) - True (5) - False (6) - True (7) - False (8) Here we got valid responses from only 4 bookies and also no writesets has more than ack number of valid responses, but RRQuorumCoverageSet's addBookieAndCheckCovered returns true in this case. Before I confirm this as bug, I need others opinion on expectation of this RRQuorumCoverage. Anyhow I added testcase to validate this logic https://github.com/reddycharan/bookkeeper/commit/9eead2212b9dc5910c8957447827edcbed4d93b6
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
