[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-749?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13982914#comment-13982914
 ] 

Ivan Kelly commented on BOOKKEEPER-749:
---------------------------------------

I would keep them separate. Conceptually they are separate, so the should be 
separate in the api also. Really locking should have gone into a separate api 
from the start, it was just convenient to have it in the same place.

> 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)

Reply via email to