dao-jun commented on PR #22560: URL: https://github.com/apache/pulsar/pull/22560#issuecomment-2963132825
> @dao-jun This PR will be a good foundation to switching to use BookKeeper's [BP-62 Batch Read API](https://github.com/apache/bookkeeper/blob/master/site3/website/src/pages/bps/BP-62-new-API-for-batched-reads.md) (introduced in [BK 4.17.0](https://bookkeeper.apache.org/release-notes/#4170)) in Pulsar. One of the gotchas of BP-62 is that a batch read might not return all entries even if there are remaining entries to be read. It seems that it would play nicely with the changes that are made in this PR so that the next read could continue to retry and read more. Would you be interested in implementing BP-62 support in Pulsar after this current PR has been merged? @lhotari Yes, I'd like to handle the case. And I created a PR to support Bookie batch read about a year ago, it's not finished yet. https://github.com/dao-jun/pulsar/pull/13 -- 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: commits-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
