----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13248/#review24668 -----------------------------------------------------------
giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/DoubleMatrix.java <https://reviews.apache.org/r/13248/#comment48777> Why are we creating rows of size numRows? Shouldn't we either default-construct the vectors or pass numColumns? - Alessandro Presta On Aug. 5, 2013, 6:25 p.m., Herald Kllapi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13248/ > ----------------------------------------------------------- > > (Updated Aug. 5, 2013, 6:25 p.m.) > > > Review request for giraph. > > > Repository: giraph-git > > > Description > ------- > > In applications where a matrix is needed, is not efficient to have an > aggregator per entry. This update provides the same functionality with an > aggregator per matrix row. > > > Diffs > ----- > > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/DoubleMatrix.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/DoubleMatrixSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/DoubleVector.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/DoubleVectorSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/FloatMatrix.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/FloatMatrixSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/FloatVector.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/FloatVectorSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/IntMatrix.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/IntMatrixSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/IntVector.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/IntVectorSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/LongMatrix.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/LongMatrixSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/LongVector.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/LongVectorSumAggregator.java > PRE-CREATION > > giraph-core/src/main/java/org/apache/giraph/aggregators/matrix/package-info.java > PRE-CREATION > > giraph-core/src/test/java/org/apache/giraph/aggregators/matrix/TestDoubleMatrix.java > PRE-CREATION > > giraph-core/src/test/java/org/apache/giraph/aggregators/matrix/TestFloatMatrix.java > PRE-CREATION > > giraph-core/src/test/java/org/apache/giraph/aggregators/matrix/TestIntMatrix.java > PRE-CREATION > > giraph-core/src/test/java/org/apache/giraph/aggregators/matrix/TestLongMatrix.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/13248/diff/ > > > Testing > ------- > > We provide test classes to test the functionality. > > > Thanks, > > Herald Kllapi > >
