showuon opened a new pull request, #15474:
URL: https://github.com/apache/kafka/pull/15474

   Fix `getOffsetByMaxTimestamp` for compressed records.
   
   This PR adds:
   1. For inPlaceAssignment case, compute the correct offset for maxTimestamp 
when traversing the batch records, and set to `ValidationResult` in the end, 
instead of setting to last offset always.
   2. For not inPlaceAssignment, set the offsetOfMaxTimestamp for the log 
create time, like non-compressed, and inPlaceAssignment cases, instead of 
setting to last offset always.
   3. Add tests to verify the fix.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to