massakam opened a new pull request, #3356:
URL: https://github.com/apache/bookkeeper/pull/3356

   ### Motivation
   
   With BP-41, the BookKeeper client now needs a request to ZooKeeper to 
resolve the address from each bookie ID.
   
   In the following document, there is a description of the flag 
`enableBookieAddressResolver` for disabling this feature by regarding the 
bookie ID as an address or hostname, but it seems that this has not been 
implemented yet.
   
https://github.com/apache/bookkeeper/blob/master/site3/website/src/pages/bps/BP-41-bookieid.md
   
   I implemented this because I want this flag to reduce the number of requests 
to ZK.
   
   ### Changes
   
   Added a flag named `enableBookieAddressResolver` to the client 
configuration. If this flag is false, use `BookieAddressResolverDisabled` 
instead of `DefaultBookieAddressResolver` as the address resolver for bookies. 
`BookieAddressResolverDisabled` regards a bookie ID to be in legacy format, 
i.e. "address:port" or "hostname:port", and returns the address of that bookie 
without access to ZK.
   
   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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to