[ 
https://issues.apache.org/jira/browse/KAFKA-671?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13566222#comment-13566222
 ] 

Sriram Subramanian commented on KAFKA-671:
------------------------------------------

The issue is that even if we pass only the required fields from produceRequest 
to DelayedProduce, we also pass the actual Request itself to delayedProduce 
which is used in multiple places. Now that contains the requestObj and hence 
there is a non zero reference to it still. Further, when sending the response 
we depend on the requestObj at multiple places in updateRequestMetrics, one of 
them being the requirement to log the complete request. We would have to do 
some non trivial changes to be able to get rid of the request object completely 
which would probably need to wait till a later time.
                
> 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
>              Labels: bugs, p1
>             Fix For: 0.8.1
>
>         Attachments: outOfMemFix-v1.patch, outOfMemFix-v2.patch, 
> outOfMemFix-v2-rebase.patch, outOfMemFix-v3.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

Reply via email to