I screwed this up. Yeah, sorry. Deleted it.
Avery
On 5/7/13 3:37 PM, Nitay Joffe wrote:
Avery,
Did you really mean to push these to heads/origin/trunk (not heads/trunk)?
If you look at the branches we now have origin/trunk and origin/origin/trunk
because of this.
- Nitay
On May 6, 2013, at 6:06 PM, [email protected] wrote:
GIRAPH-657: Remove unused reuseIncomingEdgeObjects option.
Project: http://git-wip-us.apache.org/repos/asf/giraph/repo
Commit: http://git-wip-us.apache.org/repos/asf/giraph/commit/81425f99
Tree: http://git-wip-us.apache.org/repos/asf/giraph/tree/81425f99
Diff: http://git-wip-us.apache.org/repos/asf/giraph/diff/81425f99
Branch: refs/heads/origin/trunk
Commit: 81425f99c4fe4a59846f15e7edb13ff1993b2a7b
Parents: 61ef4d2
Author: Alessandro Presta <[email protected]>
Authored: Mon May 6 12:06:56 2013 -0700
Committer: Avery Ching <[email protected]>
Committed: Mon May 6 15:04:45 2013 -0700
----------------------------------------------------------------------
CHANGELOG | 2 ++
.../apache/giraph/conf/GiraphConfiguration.java | 9 ---------
.../org/apache/giraph/conf/GiraphConstants.java | 9 ---------
3 files changed, 2 insertions(+), 18 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/giraph/blob/81425f99/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index 6439581..8a2b9eb 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,8 @@
Giraph Change Log
Release 1.0.1 - unreleased
+ GIRAPH-657: Remove unused reuseIncomingEdgeObjects option (apresta)
+
GIRAPH-592: YourKit profiler (nitay)
GIRAPH-618: Website Documentation: Aggregators (and sharded aggregators)
http://git-wip-us.apache.org/repos/asf/giraph/blob/81425f99/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java
----------------------------------------------------------------------
diff --git
a/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java
b/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java
index 48f3d4b..8d74626 100644
--- a/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java
+++ b/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java
@@ -901,15 +901,6 @@ public class GiraphConfiguration extends Configuration
}
/**
- * Check if we can reuse incoming edge objects.
- *
- * @return True iff we can reuse incoming edge objects.
- */
- public boolean reuseIncomingEdgeObjects() {
- return GiraphConstants.REUSE_INCOMING_EDGE_OBJECTS.get(this);
- }
-
- /**
* Get the local hostname on the given interface.
*
* @return The local hostname
http://git-wip-us.apache.org/repos/asf/giraph/blob/81425f99/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java
----------------------------------------------------------------------
diff --git
a/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java
b/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java
index 0067c25..54a40b7 100644
--- a/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java
+++ b/giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java
@@ -421,15 +421,6 @@ public interface GiraphConstants {
new IntConfOption("giraph.maxMutationsPerRequest", 100);
/**
- * Whether we should reuse the same Edge object when adding edges from
- * requests.
- * This works with edge storage implementations that don't keep references
- * to the input Edge objects (e.g., ByteArrayVertex).
- */
- BooleanConfOption REUSE_INCOMING_EDGE_OBJECTS =
- new BooleanConfOption("giraph.reuseIncomingEdgeObjects", false);
-
- /**
* Use message size encoding (typically better for complex objects,
* not meant for primitive wrapped messages)
*/