You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2018/11/08 11:33:01 UTC
[GitHub] ivankelly opened a new pull request #1797: Move version out of
LedgerMetadata
ivankelly opened a new pull request #1797: Move version out of LedgerMetadata
URL: https://github.com/apache/bookkeeper/pull/1797
The version is a property of the metadata store and not of the
LedgerMetadata object itself. Putting it in the LedgerMetadata forces
that LedgerMetadata to be mutable, as it needs to be updated before
being sent to the LedgerManager.
This change moves version out of LedgerMetadata and modifies the
LedgerManager to use Versioned<LedgerMetadata>.
This is a large change, but almost purely mechanical.
TestHttpService has been modified to remove a preexisting flake in a
test which had to be modified for this change.
Master issue: #281
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services