Review Board 1.7.22


BOOKKEEPER-152 Can't recover a ledger whose current ensemble contain failed bookie.

Review Request #3737 - Created Feb. 2, 2012 and updated

Ivan Kelly
BOOKKEEPER-152
Reviewers
bookkeeper
bookkeeper-git
Proposed fix ensures that at least one of each quorum replies to ReadLastConfirmed.

Refactors code a bit to make the read last confirmed common for recovery and standalone read last confirmed.

The bug here was actually that we were waiting for quorumSize responses, from the bookies, when really all we need to get a response from one bookie in each possible quorum. in the 2/2 case as above this means only 1 bookie need response.

There's a fix for the timeouts and an improvement in fencing which fixing this uncovered.

 
Review request changed
Updated (Feb. 13, 2012, 12:16 p.m.)