[
https://issues.apache.org/jira/browse/KAFKA-671?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13558462#comment-13558462
]
Sriram Subramanian commented on KAFKA-671:
------------------------------------------
I think to do this right we should limit the change to produceRequest instead
of getting rid of the requestobj completely. The reason is as you mention you
would need to cache a bunch of fields from requestObj to be used in
updateRequestMetrics while sending the response. This can be two fields today
but going forward you may need more fields and this would need to be constantly
maintained. I think if produceRequest handles data being null, the fix would be
more robust and isolates the changes within produce request.
> DelayedProduce requests should not hold full producer request data
> ------------------------------------------------------------------
>
> Key: KAFKA-671
> URL: https://issues.apache.org/jira/browse/KAFKA-671
> Project: Kafka
> Issue Type: Bug
> Affects Versions: 0.8
> Reporter: Joel Koshy
> Assignee: Sriram Subramanian
> Priority: Blocker
> Fix For: 0.8.1
>
> Attachments: outOfMemFix-v1.patch
>
>
> Per summary, this leads to unnecessary memory usage.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira