[ https://issues.apache.org/jira/browse/BOOKKEEPER-749?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13982899#comment-13982899 ]
Ivan Kelly commented on BOOKKEEPER-749: --------------------------------------- I would put async notification into separate apis then, like notifyUnderreplicationChange() and notifyLockStatusChange(). The former may be difficult to implement due to nesting though. > Improve LedgerUnderreplicationManager#getLedgerToRereplicate() > -------------------------------------------------------------- > > Key: BOOKKEEPER-749 > URL: https://issues.apache.org/jira/browse/BOOKKEEPER-749 > Project: Bookkeeper > Issue Type: Improvement > Components: bookkeeper-auto-recovery > Reporter: Rakesh R > Assignee: Rakesh R > Attachments: 00001-BOOKKEEPER-749-interface.patch > > > As per the discussion in BOOKKEEPER-733, > LedgerUnderreplicationManager#getLedgerToRereplicate having multiple > resposibilities like : getUrLedger + acquireLock. Actually the idea of this > JIRA is to make the interface better. > Please see the discussion thread: > https://issues.apache.org/jira/browse/BOOKKEEPER-733?focusedCommentId=13963072&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13963072 -- This message was sent by Atlassian JIRA (v6.2#6252)