[KAFKA-10841] LogReadResult should be able to converted to FetchPartitionData

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.8.0
• Labels: newbie
• Assignee: GeordieMai
• Reporter: Chia-Ping Tsai
• Created: 11/Dec/20 05:34
• Updated: 19/Jan/21 00:56
• Resolved: 19/Jan/21 00:56

Description

There are duplicate code which try to convert LogReadResult to FetchPartitionData. It seems to me the duplicate code can be eliminated by moving the conversion to LogReadResult.

occurrence 1: https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/server/ReplicaManager.scala#L1076

occurrence 2:
https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/server/DelayedFetch.scala#L189

Issue Links

• links to: GitHub Pull Request #9743