----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10598/#review19401 -----------------------------------------------------------
This is perfect now, +1 giraph-core/src/main/java/org/apache/giraph/utils/EdgeIterables.java <https://reviews.apache.org/r/10598/#comment40115> Nit: you may be able to remove the parentheses and maybe fit in one line. - Alessandro Presta On April 18, 2013, 6:45 p.m., Maja Kabiljo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10598/ > ----------------------------------------------------------- > > (Updated April 18, 2013, 6:45 p.m.) > > > Review request for giraph. > > > Description > ------- > > Compute OutEdges can for example be immutable, or have better performance > when adding all edges than when adding one by one. > > > This addresses bug giraph-636. > https://issues.apache.org/jira/browse/giraph-636 > > > Diffs > ----- > > giraph-core/src/main/java/org/apache/giraph/edge/EdgeStore.java 81044bd > giraph-core/src/main/java/org/apache/giraph/edge/HashMapEdges.java fc23cf0 > giraph-core/src/main/java/org/apache/giraph/edge/HashMultimapEdges.java > abe6151 > giraph-core/src/main/java/org/apache/giraph/edge/IntNullArrayEdges.java > 9867771 > giraph-core/src/main/java/org/apache/giraph/edge/LongDoubleArrayEdges.java > 45d182e > > giraph-core/src/main/java/org/apache/giraph/edge/LongDoubleHashMapEdges.java > c33200d > giraph-core/src/main/java/org/apache/giraph/edge/LongNullArrayEdges.java > d753588 > giraph-core/src/main/java/org/apache/giraph/edge/LongNullHashSetEdges.java > ef0302c > giraph-core/src/main/java/org/apache/giraph/utils/EdgeIterables.java > e945d7b > > Diff: https://reviews.apache.org/r/10598/diff/ > > > Testing > ------- > > mvn clean verify > Application run which requires this kind of initialization > > > Thanks, > > Maja Kabiljo > >
