Repository: giraph Updated Branches: refs/heads/trunk f93031877 -> 7b4a5b7fa
[GIRAPH-1005] Reorder MessageEncodeAndStoreType Summary: It is useful to have ordering among MessageEncodeAndStoreType, from one with least overhead and least optimized for good partitioning, to one with most overhead and most optimized for good partitioning. Test Plan: mvn clean install -Phadoop_facebook Reviewers: maja.kabiljo, dionysis.logothetis Reviewed By: dionysis.logothetis Differential Revision: https://reviews.facebook.net/D37959 Project: http://git-wip-us.apache.org/repos/asf/giraph/repo Commit: http://git-wip-us.apache.org/repos/asf/giraph/commit/7b4a5b7f Tree: http://git-wip-us.apache.org/repos/asf/giraph/tree/7b4a5b7f Diff: http://git-wip-us.apache.org/repos/asf/giraph/diff/7b4a5b7f Branch: refs/heads/trunk Commit: 7b4a5b7fabac83a32ff62fda2108f860f2caaac4 Parents: f930318 Author: Igor Kabiljo <[email protected]> Authored: Thu Apr 30 16:41:48 2015 -0700 Committer: Igor Kabiljo <[email protected]> Committed: Fri May 1 14:02:47 2015 -0700 ---------------------------------------------------------------------- .../giraph/comm/messages/MessageEncodeAndStoreType.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/giraph/blob/7b4a5b7f/giraph-core/src/main/java/org/apache/giraph/comm/messages/MessageEncodeAndStoreType.java ---------------------------------------------------------------------- diff --git a/giraph-core/src/main/java/org/apache/giraph/comm/messages/MessageEncodeAndStoreType.java b/giraph-core/src/main/java/org/apache/giraph/comm/messages/MessageEncodeAndStoreType.java index 7a5b702..3da0985 100644 --- a/giraph-core/src/main/java/org/apache/giraph/comm/messages/MessageEncodeAndStoreType.java +++ b/giraph-core/src/main/java/org/apache/giraph/comm/messages/MessageEncodeAndStoreType.java @@ -24,18 +24,18 @@ package org.apache.giraph.comm.messages; */ public enum MessageEncodeAndStoreType { /** - * Use message-store which is based on list of pointers to encoded messages + * Use a byte-array to store messages for each partition */ - POINTER_LIST_PER_VERTEX(true), + BYTEARRAY_PER_PARTITION(false), /** * Extract a byte array per partition from one message to many ids encoding * and then store */ EXTRACT_BYTEARRAY_PER_PARTITION(true), /** - * Use a byte-array to store messages for each partition + * Use message-store which is based on list of pointers to encoded messages */ - BYTEARRAY_PER_PARTITION(false); + POINTER_LIST_PER_VERTEX(true); /** Can use one message to many ids encoding? */ private final boolean oneMessageToManyIdsEncoding;
