satishd commented on code in PR #13535: URL: https://github.com/apache/kafka/pull/13535#discussion_r1171250580
########## core/src/main/scala/kafka/server/ReplicaManager.scala: ########## @@ -1118,9 +1122,13 @@ class ReplicaManager(val config: KafkaConfig, responseCallback: Seq[(TopicIdPartition, FetchPartitionData)] => Unit ): Unit = { // check if this fetch request can be satisfied right away - val logReadResults = readFromLocalLog(params, fetchInfos, quota, readFromPurgatory = false) + val logReadResults = readFromLog(params, fetchInfos, quota, readFromPurgatory = false) var bytesReadable: Long = 0 var errorReadingData = false + + // The 1st topic-partition that has to be read from remote storage + var remoteFetchInfo: Optional[RemoteStorageFetchInfo] = Optional.empty() Review Comment: As I already called out in this PR description, that it is followed up with a PR. We will describe the config on different options with respective scenarios. The default value will be to fetch from multiple partitions as it does with local log segments. -- 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