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

   In this PR, we noticed failed tests caused by the verification of log start 
offset and local log start offset. After investigation: 
https://github.com/apache/kafka/pull/14347#issuecomment-1717110340
   
   After investigation, I found it's because the scala 2.12 cannot recognize 
the `override` method of `maybeWaitForAtLeastOneSegmentUpload` since it's using 
`varargs` in scala. I think there must be some bugs/gaps between java/scala 
that causes these issue. We can fix it by not using `varargs`, instead, using 
the `Seq`.
   
https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-14347/13/#showFailuresLink
   
   This PR adds back the log start offset and local log start offset 
verification, and make sure all tests passed.
   
   ### 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