giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Herald Kllapi" <heral...@gmail.com>
Subject Review Request 14025: Dense matrix aggregator implementation.
Date Fri, 06 Sep 2013 23:01:54 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14025/
-----------------------------------------------------------

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. This implementation
uses an array per row and is efficient when the matrices are dense.


Diffs
-----

  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/DenseMatrixSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/DoubleDenseMatrix.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/DoubleDenseMatrixSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/DoubleDenseVector.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/DoubleDenseVectorSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/FloatDenseMatrix.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/FloatDenseMatrixSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/FloatDenseVector.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/FloatDenseVectorSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/IntDenseMatrix.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/IntDenseMatrixSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/IntDenseVector.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/IntDenseVectorSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/LongDenseMatrix.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/LongDenseMatrixSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/LongDenseVector.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/LongDenseVectorSumAggregator.java
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/aggregators/denseMatrix/package-info.java PRE-CREATION

  giraph-core/src/test/java/org/apache/giraph/aggregators/denseMatrix/TestDoubleDenseMatrix.java
PRE-CREATION 
  giraph-core/src/test/java/org/apache/giraph/aggregators/denseMatrix/TestFloatDenseMatrix.java
PRE-CREATION 
  giraph-core/src/test/java/org/apache/giraph/aggregators/denseMatrix/TestIntDenseMatrix.java
PRE-CREATION 
  giraph-core/src/test/java/org/apache/giraph/aggregators/denseMatrix/TestLongDenseMatrix.java
PRE-CREATION 

Diff: https://reviews.apache.org/r/14025/diff/


Testing
-------


Thanks,

Herald Kllapi


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message