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

Ivan Kelly commented on BOOKKEEPER-299:
---------------------------------------

LedgerFragmentReplicator should be as it is, but all methods on it should be 
package private (or private). Then BookKeeperAdmin can call into 
LedgerFragmentReplicator to do the actual rereplication, so yes, 
LedgerFragmentReplicator is effectively a helper class. I'd like to move as 
much code as possible out of BookKeeperAdmin.
                
> Provide LedgerFragmentReplicator which should replicate the fragments found 
> from LedgerChecker
> ----------------------------------------------------------------------------------------------
>
>                 Key: BOOKKEEPER-299
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-299
>             Project: Bookkeeper
>          Issue Type: Sub-task
>          Components: bookkeeper-client, bookkeeper-server
>    Affects Versions: 4.2.0
>            Reporter: Uma Maheswara Rao G
>            Assignee: Uma Maheswara Rao G
>         Attachments: BOOKKEEPER-299.patch
>
>
> Replication worker requires LedgerFragmentReplicator for replicating the 
> actula fragments found from Ledger checker.
> Most of the fragment replication code available in BookKeeperAdmin. We can 
> refactor it to LedgerFragmentReplicator and use it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to