[BOOKKEEPER-900] read only bookie runs replicator and does not release the under replicated lock after failing

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.3.2
• Fix Version/s: 4.4.0
• Component/s: bookkeeper-auto-recovery
• Assignee: Siddharth Boobna
• Reporter: Siddharth Boobna
• Created: 29/Feb/16 03:46
• Updated: 16/May/16 21:47
• Resolved: 08/Mar/16 06:16

Description

ReplicationWorker throws BkWriteOnReadOnlyBookieException here: https://github.com/apache/bookkeeper/blob/master/bookkeeper-server/src/main/java/org/apache/bookkeeper/replication/ReplicationWorker.java#L255
and does not release the under replicated lock.

Solution:
Release the under replicated lock before throwing BkWriteOnReadOnlyBookieException and make the replication worker wait till the bookie becomes writable again.