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

Maja Kabiljo commented on GIRAPH-839:
-------------------------------------

Thanks, +1

> NettyWorkerAggregatorRequestProcessor tries to reuse request objects
> --------------------------------------------------------------------
>
>                 Key: GIRAPH-839
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-839
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Pavan Kumar
>            Assignee: Pavan Kumar
>         Attachments: GIRAPH-839.patch
>
>
> I was working on upgrading netty to 4.0.14.Final. It is mostly done, after 
> some polishing I will submit a patch for that soon.
> While working on the above, I discovered a bug in 
> NettyWorkerAggregatorRequestProcessor. The distributeAggregators method is 
> reusing SendAggregatorsToWorkerRequest objects, which is incorrect since each 
> task has a different task id & (potentially a different) request id. It does 
> not cause any problem with current netty but in the new version I guess due 
> to delayed serialization the object cannot be reused.
> Anyway, moving it inside the for loop does not cause any issues with current 
> netty as well.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to