chia7712 commented on code in PR #15904: URL: https://github.com/apache/kafka/pull/15904#discussion_r1599412941
########## core/src/test/scala/unit/kafka/log/LogValidatorTest.scala: ########## @@ -488,8 +496,11 @@ class LogValidatorTest { } assertEquals(now + 1, validatingResults.maxTimestampMs, s"Max timestamp should be ${now + 1}") - assertEquals(2, validatingResults.shallowOffsetOfMaxTimestamp, - "Shallow offset of max timestamp should be 2") + + // Both V2 and V1 has single branch in the record when compression is enable, and hence their shallow OffsetOfMaxTimestamp + // is the last offset of the single branch + assertEquals(1, records.batches().asScala.size) Review Comment: @vincent81jiang you are totally right. I have filed PR (https://github.com/apache/kafka/pull/15948) to fix that -- 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