This is an automated email from the ASF dual-hosted git repository.

ijuma pushed a commit to branch 4.0
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/4.0 by this push:
     new 699db8a99f2 MINOR: fix LogOffsetTest in 4.0 (#18495)
699db8a99f2 is described below

commit 699db8a99f2f6a2b01507d10cfc8d01e62840622
Author: TaiJuWu <tjwu1...@gmail.com>
AuthorDate: Mon Jan 13 05:52:12 2025 +0800

    MINOR: fix LogOffsetTest in 4.0 (#18495)
    
    Convert Some to Optional
    
    Reviewers: Ismael Juma <ism...@juma.me.uk>
---
 core/src/test/scala/unit/kafka/server/LogOffsetTest.scala | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/src/test/scala/unit/kafka/server/LogOffsetTest.scala 
b/core/src/test/scala/unit/kafka/server/LogOffsetTest.scala
index 1b6564c0505..dc3e7d7c6ab 100755
--- a/core/src/test/scala/unit/kafka/server/LogOffsetTest.scala
+++ b/core/src/test/scala/unit/kafka/server/LogOffsetTest.scala
@@ -82,7 +82,7 @@ class LogOffsetTest extends BaseRequestTest {
     log.deleteOldSegments()
 
     val offset = 
log.fetchOffsetByTimestamp(ListOffsetsRequest.LATEST_TIMESTAMP).timestampAndOffsetOpt.map(_.offset)
-    assertEquals(Some(20L), offset)
+    assertEquals(Optional.of(20L), offset)
 
     TestUtils.waitUntilTrue(() => isLeaderLocalOnBroker(topic, 
topicPartition.partition, broker),
       "Leader should be elected")
@@ -149,7 +149,7 @@ class LogOffsetTest extends BaseRequestTest {
     log.flush(false)
 
     val offset = 
log.fetchOffsetByTimestamp(ListOffsetsRequest.LATEST_TIMESTAMP).timestampAndOffsetOpt.map(_.offset)
-    assertEquals(Some(20L), offset)
+    assertEquals(Optional.of(20L), offset)
 
     TestUtils.waitUntilTrue(() => isLeaderLocalOnBroker(topic, 0, broker),
       "Leader should be elected")
@@ -219,7 +219,7 @@ class LogOffsetTest extends BaseRequestTest {
     log.flush(false)
 
     val offset = 
log.fetchOffsetByTimestamp(ListOffsetsRequest.EARLIEST_TIMESTAMP).timestampAndOffsetOpt.map(_.offset)
-    assertEquals(Some(0L), offset)
+    assertEquals(Optional.of(0L), offset)
 
     TestUtils.waitUntilTrue(() => isLeaderLocalOnBroker(topic, 
topicPartition.partition, broker),
       "Leader should be elected")

Reply via email to