iit2009060 commented on code in PR #14482:
URL: https://github.com/apache/kafka/pull/14482#discussion_r1346817006


##########
core/src/test/scala/unit/kafka/log/remote/RemoteIndexCacheTest.scala:
##########
@@ -541,6 +542,131 @@ class RemoteIndexCacheTest {
     verifyFetchIndexInvocation(count = 1)
   }
 
+  @Test
+  def testIndexFileAlreadyExistOnDiskButNotInCache(): Unit = {

Review Comment:
   Yes @showuon  , this can happen in real case
   1. Call getIndexEntry 
   2. Offset index is successfully cached 
   3. But TimeIndex received from remote storage is corrupted 
   4. getIndexEntry throws Corrupt Exception
   But it leaves the offset index file on disk in remote cache directory. 



-- 
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