imavishek commented on PR #23246:
URL: https://github.com/apache/kafka/pull/23246#issuecomment-5386941804

   @kamalcph @showuon would you be able to take a look at this when you have a 
moment? It's a small follow-on to KAFKA-19995 and KAFKA-20732: on 
low-throughput partitions a single-record segment (baseOffset == endOffset == 
highestOffsetInRemoteStorage) is already copied to remote, but the `>=` filter 
in `onlyLocalLogSegments{Count,Size}` still counts it as local, leaving 
RemoteCopyLagSegments latched at a phantom 1. The fix switches to a strict `>`; 
included a regression test that fails on the old code. Thanks!


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

Reply via email to