kowshik commented on pull request #10684: URL: https://github.com/apache/kafka/pull/10684#issuecomment-849263208
@junrao Thanks for the review and for catching this issue! I've fixed the build now. The issue was that the code in `Log.scala` used the `Iterator.nextOption()`[API](https://www.scala-lang.org/api/2.13.0/scala/collection/Iterator.html#nextOption():Option[A]) that was introduced only in scala v2.13, so the build failed for scala v2.12. I have now also added unit tests for the newly introduced `LogSegments.higherSegments()` API. The PR is ready for review again. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org