[
https://issues.apache.org/jira/browse/BOOKKEEPER-315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13458621#comment-13458621
]
Ivan Kelly commented on BOOKKEEPER-315:
---------------------------------------
splitToSubFragments is incorrect.
+ long numberOfEntriesToReplicate = lastEntryId - firstEntryId;
should be (lastEntryId - firstEntryId) + 1
typo in maxAllowedEntriesInLogicalFrgament. name is too long anyhow.
entriesPerSubFragment is better
getMaxAllowedEtriesPerFragmentInRereplication has a typo and is too long.
Change to getReplicationBatchSize()
In general, I think this splitting should be in LedgerFragmentReplicator,
rather than BookKeeperAdmin. This way you could avoid the need to expose the
update of ledger metadata. It will make control flow a little more complex as
you'll have to initiate replication on the next subfragment from the callback
of the current, but i think it would encapsulate things a bit better.
> Ledger entries should be replicated sequentially instead of parallel.
> ---------------------------------------------------------------------
>
> Key: BOOKKEEPER-315
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-315
> Project: Bookkeeper
> Issue Type: Sub-task
> Components: bookkeeper-auto-recovery, bookkeeper-client
> Affects Versions: 4.2.0
> Reporter: Uma Maheswara Rao G
> Assignee: Uma Maheswara Rao G
> Attachments: BOOKKEEPER-315.patch
>
>
> Currently BookKeeperAdmin will copy the entries parallel.
> This may create more load on the servers. To avoid that, we can refactor the
> BKAdmin code to copy the entries sequential.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
