apoorvmittal10 commented on PR #23468: URL: https://github.com/apache/kafka/pull/23468#issuecomment-5679843185
@Shekharrajak Thanks for the PR, can you please help me understand the benefits of the change? Though it makes sense to have a single call but I am trying to understand is there any real benefit seen in the tests you benchmarked? The idea is to keep `nextFetchOffset()` call very lean i.e. least computation hence the variable `nextFetchOffset` is updated. Which means if nothing changed in the mean time of subsequent calls then no scan will be performed. Do you think this PR can further improve the performance? -- 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]
