You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@giraph.apache.org by Bo Wang <da...@gmail.com> on 2012/04/23 08:15:57 UTC

Review Request: Change coarse-grain lock to fine-grain lock by pre-populating the message map so that each entry for a vertex can be locked individually and the hazard condition is also avoided.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4840/
-----------------------------------------------------------

Review request for giraph.


Summary
-------

Improve concurrency of putMsg / putMsgList.


This addresses bug GIRAPH-185.
    https://issues.apache.org/jira/browse/GIRAPH-185


Diffs
-----

  http://svn.apache.org/repos/asf/incubator/giraph/trunk/src/main/java/org/apache/giraph/comm/BasicRPCCommunications.java 1328747 
  http://svn.apache.org/repos/asf/incubator/giraph/trunk/src/main/java/org/apache/giraph/comm/WorkerCommunications.java 1328747 
  http://svn.apache.org/repos/asf/incubator/giraph/trunk/src/main/java/org/apache/giraph/graph/BspServiceWorker.java 1328747 

Diff: https://reviews.apache.org/r/4840/diff


Testing
-------

All test passed. Wait for review. Thanks.


Thanks,

Bo