giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Maja Kabiljo (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GIRAPH-928) SendWorkerMessagesRequest should not store messages into PointerList based messagestores
Date Fri, 11 Jul 2014 18:42:07 GMT

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

Maja Kabiljo commented on GIRAPH-928:
-------------------------------------

+1

> SendWorkerMessagesRequest should not store messages into PointerList based messagestores
> ----------------------------------------------------------------------------------------
>
>                 Key: GIRAPH-928
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-928
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Pavan Kumar
>            Assignee: Pavan Kumar
>            Priority: Minor
>         Attachments: GIRAPH-928, GIRAPH-928.patch
>
>
> While running some test jobs I realized that after GIRAPH-912, some old logic inside
SendOneMessageToManyCache can cause a situation where SendWorkerMessagesRequest try to store
messages into PointerList based messagestores. This is semantically incorrect. So the code
needs to be refactored to remove this possibility. Also adding sanity checks which can be
removed later if felt redundant.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message