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

Posted to dev@giraph.apache.org by Bingjing Zhang <je...@gmail.com> on 2013/08/07 01:17:24 UTC

Review Request 13366: Modify message store to support adding one-to-all messages

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

Review request for giraph.


Bugs: GIRAPH-701
    https://issues.apache.org/jira/browse/GIRAPH-701


Repository: giraph-git


Description
-------

modify message store to support adding one-to-all messages


Diffs
-----

  giraph-core/src/main/java/org/apache/giraph/comm/ServerData.java 39bf504 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/InMemoryMessageStoreFactory.java 0cdfb73 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/MessageStore.java 2af7642 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/SimpleMessageStore.java 13292a2 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/out_of_core/DiskBackedMessageStore.java 346e3b3 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/primitives/IntByteArrayMessageStore.java 597e7af 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/primitives/IntFloatMessageStore.java a193fb9 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/primitives/LongByteArrayMessageStore.java 3fe6356 
  giraph-core/src/main/java/org/apache/giraph/comm/messages/primitives/LongDoubleMessageStore.java 96ed5b4 
  giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerServer.java 3473de1 
  giraph-core/src/main/java/org/apache/giraph/comm/requests/SendWorkerOneToAllMessagesRequest.java 8745adb 
  giraph-core/src/test/java/org/apache/giraph/comm/TestMessageStores.java eb2497b 
  giraph-core/src/test/java/org/apache/giraph/comm/messages/TestIntFloatPrimitiveMessageStores.java a8f6f70 
  giraph-core/src/test/java/org/apache/giraph/comm/messages/TestLongDoublePrimitiveMessageStores.java 0659260 
  giraph-core/src/test/java/org/apache/giraph/utils/MockUtils.java 97e88f8 

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


Testing
-------

test with "mvn clean install"


Thanks,

Bingjing Zhang