----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13248/ -----------------------------------------------------------
(Updated Aug. 6, 2013, 6:38 a.m.) Review request for giraph. Changes ------- Added MatrixSumAggregator and fixed some style errors. 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 (updated) ----- 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/MatrixSumAggregator.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
