eolivelli commented on PR #17241: URL: https://github.com/apache/pulsar/pull/17241#issuecomment-1231747647
@zymap @codelipenghui @Jason918 @lhotari I have added one last step to the patch and now it is really ready: - added the ability to reuse pending reads that partially overlap with the requested range - refactored the new code to a dedicated class, in order to reduce the complexity of the code (and to ease testability) I tried to do something more sophisticated, like reusing only overlapping pending reads only if the amount of overlapping entries is big enough, but in my testing I have seen that it seems always a good idea to reuse pending reads. -- 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]
