jeqo commented on code in PR #13535: URL: https://github.com/apache/kafka/pull/13535#discussion_r1165444921
########## 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: I understand a new PR will come to overcome this, but could we provide further context (on the source code or PR) about the implications of using the first topic-partition only? -- 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