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

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

I haven't looked very deep into this yet, but one suggestion I have, is to make 
LedgerFragmentReplicator#replicate take a LedgerFragment (Added in BK-247) and 
a newBookie rather than the long parameter list it has now.

                
> 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, BOOKKEEPER-299.patch, 
> 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