[GitHub] eolivelli opened a new pull request #1572: WIP - Use readExplicitLAC instead of readEntry in order to get current LastAddConfirmed

[GitBox <gi...@apache.org>]

eolivelli opened a new pull request #1572: WIP - Use readExplicitLAC instead of readEntry in order to get current LastAddConfirmed
URL: https://github.com/apache/bookkeeper/pull/1572
 
 
   ### Motivation
   
   Readers have to explicitly call readExplicitLastConfirmed in order to leverage ExplicitLAC.
   I would like to have a unified way to read LastAddConfirmed.
   
   
   ### Changes
   This change makes the client use internally readExplicitLastConfirmed instead of using readLastAddConfirmed.
   We need to introduce something similar to tryReadLastAddConfirmed in order to make it consistent  with readLastAddConfirmed.
   
   This PR is only a proof-of-concept for early review, it is not intended to be merged to BK code base as it is in the current form.
   
   It lacks supporto for Long Poll Reads, expecially we have to listen on ExplicitLAC modifications on the bookie side in order to wake up the readers even in case of setExplicitLAC from the writer.

----------------------------------------------------------------
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