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 2020/09/08 17:16:52 UTC

[GitHub] [bookkeeper] eolivelli opened a new pull request #2412: BP-41 make bookieId configurable

eolivelli opened a new pull request #2412:
URL: https://github.com/apache/bookkeeper/pull/2412


   ### Motivation
   This is the bookie side implementation of BP-41
   
   ### Changes
   - add new `bookieId` configuration option
   - implement Bookie#getBookieId
   - add test cases
   - make all BookKeeperClusterTestCase to use a random UUID as bookieId in order to increase code coverage
   - add a default BookieServiceInfo provider in tests
   - fix Auditor to use BookieId and not BookieSocketAddress.toBookieId()
   - use BookieId for LocalTransport addresses
   - in Cookie.java rename BookieHost to BookieId (only in Java code, no data format changes)
   
   Master Issue: #2396
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [bookkeeper] jiazhai merged pull request #2412: BP-41 make bookieId configurable

Posted by GitBox <gi...@apache.org>.

jiazhai merged pull request #2412:
URL: https://github.com/apache/bookkeeper/pull/2412


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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