ijuma commented on pull request #9758: URL: https://github.com/apache/kafka/pull/9758#issuecomment-790614014
Great to see this merged. @chia7712 Will you follow up with the items we split into separate JIRAs? I think the 3 key ones are: 1. Remove unnecessary copying, if possible, in KafkaApis 2. Remove the lazily populated map in FetchResponse 3. I think you had an idea for removing `FetchResponse.toMessage` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org