nandini12396 commented on code in PR #21089:
URL: https://github.com/apache/kafka/pull/21089#discussion_r2625172999


##########
storage/src/main/java/org/apache/kafka/server/log/remote/storage/RemoteLogManagerConfig.java:
##########
@@ -92,6 +92,13 @@ public final class RemoteLogManagerConfig {
             "from remote storage in the local storage.";
     public static final long 
DEFAULT_REMOTE_LOG_INDEX_FILE_CACHE_TOTAL_SIZE_BYTES = 1024 * 1024 * 1024L;
 
+    public static final String REMOTE_LOG_INDEX_FILE_CACHE_TTL_MS_PROP = 
"remote.log.index.file.cache.ttl.ms";
+    public static final String REMOTE_LOG_INDEX_FILE_CACHE_TTL_MS_DOC = "The 
maximum time in milliseconds an index file entry can remain in the cache " +
+            "after its last access. After this duration, the entry will be 
evicted even if there is available space. " +
+            "This helps prevent old index files from sitting in cache 
indefinitely, especially useful for backfill workloads. " +
+            "Set to -1 to disable time-based eviction (default behavior uses 
only size-based eviction).";
+    public static final long DEFAULT_REMOTE_LOG_INDEX_FILE_CACHE_TTL_MS = -1L;

Review Comment:
   yes, makes sense, updated to 1h. Thanks



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to