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