You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@giraph.apache.org by cl...@apache.org on 2013/08/15 16:46:51 UTC

git commit: updated refs/heads/trunk to 44ae3c3

Updated Branches:
  refs/heads/trunk 8a147a30e -> 44ae3c38c


GIRAPH-740


Project: http://git-wip-us.apache.org/repos/asf/giraph/repo
Commit: http://git-wip-us.apache.org/repos/asf/giraph/commit/44ae3c38
Tree: http://git-wip-us.apache.org/repos/asf/giraph/tree/44ae3c38
Diff: http://git-wip-us.apache.org/repos/asf/giraph/diff/44ae3c38

Branch: refs/heads/trunk
Commit: 44ae3c38c5560f869623a6f286f5c657b6d9a37f
Parents: 8a147a3
Author: Claudio Martella <cl...@apache.org>
Authored: Thu Aug 15 16:46:29 2013 +0200
Committer: Claudio Martella <cl...@apache.org>
Committed: Thu Aug 15 16:46:29 2013 +0200

----------------------------------------------------------------------
 CHANGELOG                                                         | 3 +++
 .../src/main/java/org/apache/giraph/utils/ArrayListWritable.java  | 2 ++
 2 files changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/giraph/blob/44ae3c38/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index 1e5865e..e79a04d 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,9 @@
 Giraph Change Log
 
 Release 1.1.0 - unreleased
+  GIRAPH-740: ArrayListWritable object is not cleared in readFields()
+  (korlando via claudio)
+
   GIRAPH-692: Add Bracha Toueg Deadlock Detection in the examples 
   (aarmax00 via claudio)
 

http://git-wip-us.apache.org/repos/asf/giraph/blob/44ae3c38/giraph-core/src/main/java/org/apache/giraph/utils/ArrayListWritable.java
----------------------------------------------------------------------
diff --git a/giraph-core/src/main/java/org/apache/giraph/utils/ArrayListWritable.java b/giraph-core/src/main/java/org/apache/giraph/utils/ArrayListWritable.java
index 9d8f4f1..2d9928a 100644
--- a/giraph-core/src/main/java/org/apache/giraph/utils/ArrayListWritable.java
+++ b/giraph-core/src/main/java/org/apache/giraph/utils/ArrayListWritable.java
@@ -95,6 +95,8 @@ public abstract class ArrayListWritable<M extends Writable> extends ArrayList<M>
     if (this.refClass == null) {
       setClass();
     }
+
+    clear();                              // clear list before storing values
     int numValues = in.readInt();            // read number of values
     ensureCapacity(numValues);
     for (int i = 0; i < numValues; i++) {