Github user eolivelli commented on the issue:
https://github.com/apache/bookkeeper/pull/180
@sijie I did a more deep review and it is all great for me.
Do you think it would be useful to add a test for readUnconfirmedEntries ?
Actually the implementation is shared with readEntries so maybe we can live
without such new test
b.q. Just to understand better
How are you using this feature to improve speed of "tailing reads" ? I see
that the LAC is advanced without calling readLastAddConfirmed but there is no
guarantee that it will actually advance so sometimes you will need to call
readLastAddConfirmed anyway, isn't it ?.
How can you distinguish the case of dead writer/no more data/closed ledger
? by catching exceptions ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
