[
https://issues.apache.org/jira/browse/GIRAPH-259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13416307#comment-13416307
]
Maja Kabiljo commented on GIRAPH-259:
-------------------------------------
I see that when I fix this local test fails because there everybody share the
same instance of Aggregator.
The way aggregators are implemented now, in the compute method vertices don't
necessarily see the value of aggregator from previous superstep, but they also
see the modifications which other vertices from the same worker made during the
current superstep. I think this should be fixed. One way to fix it is to expose
only getValue and aggregate methods (don't give aggregator instances), and then
have two instances of aggregator behind - the one from previous superstep and
the one for current which we'll be sending later.
> TestBspBasic.testBspPageRank is broken
> --------------------------------------
>
> Key: GIRAPH-259
> URL: https://issues.apache.org/jira/browse/GIRAPH-259
> Project: Giraph
> Issue Type: Bug
> Reporter: Maja Kabiljo
> Assignee: Maja Kabiljo
>
> Test crashes on line 152 in class SimplePageRankVertex in distributed mode.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
