[ 
https://issues.apache.org/jira/browse/GIRAPH-192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13398574#comment-13398574
 ] 

Hudson commented on GIRAPH-192:
-------------------------------

Integrated in giraph-trunk-hadoop-trunk #7 (See 
[https://builds.apache.org/job/giraph-trunk-hadoop-trunk/7/])
    GIRAPH-192. Move aggregators to a separate sub-package. Contributed by Jan 
van der Lugt. (Revision 1352585)

     Result = FAILURE
jghoman : 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1352585
Files : 
* /giraph/trunk/CHANGELOG
* /giraph/trunk/src/main/java/org/apache/giraph/aggregators
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/BooleanAndAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/BooleanOrAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/BooleanOverwriteAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/DoubleAverageAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/DoubleMaxAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/DoubleMinAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/DoubleOverwriteAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/DoubleProductAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/DoubleSumAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/FloatAverageAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/FloatMaxAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/FloatMinAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/FloatOverwriteAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/FloatProductAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/FloatSumAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/IntMaxAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/IntMinAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/IntOverwriteAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/IntProductAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/IntSumAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/LongMaxAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/LongMinAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/LongOverwriteAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/LongProductAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/aggregators/LongSumAggregator.java
* /giraph/trunk/src/main/java/org/apache/giraph/aggregators/package-info.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/benchmark/RandomMessageBenchmark.java
* /giraph/trunk/src/main/java/org/apache/giraph/examples/LongSumAggregator.java
* /giraph/trunk/src/main/java/org/apache/giraph/examples/MaxAggregator.java
* /giraph/trunk/src/main/java/org/apache/giraph/examples/MinAggregator.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/examples/SimpleCheckpointVertex.java
* 
/giraph/trunk/src/main/java/org/apache/giraph/examples/SimplePageRankVertex.java
* /giraph/trunk/src/main/java/org/apache/giraph/examples/SumAggregator.java
* /giraph/trunk/src/main/java/org/apache/giraph/examples/VerifyMessage.java
* /giraph/trunk/src/test/java/org/apache/giraph/TestBspBasic.java
* /giraph/trunk/src/test/java/org/apache/giraph/aggregators
* 
/giraph/trunk/src/test/java/org/apache/giraph/aggregators/TestBooleanAggregators.java
* 
/giraph/trunk/src/test/java/org/apache/giraph/aggregators/TestDoubleAggregators.java
* 
/giraph/trunk/src/test/java/org/apache/giraph/aggregators/TestFloatAggregators.java
* 
/giraph/trunk/src/test/java/org/apache/giraph/aggregators/TestIntAggregators.java
* 
/giraph/trunk/src/test/java/org/apache/giraph/aggregators/TestLongAggregators.java

                
> Move aggregators to a separate sub-package
> ------------------------------------------
>
>                 Key: GIRAPH-192
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-192
>             Project: Giraph
>          Issue Type: Improvement
>          Components: examples
>    Affects Versions: 0.2.0
>            Reporter: Jan van der Lugt
>            Assignee: Jan van der Lugt
>            Priority: Minor
>             Fix For: 0.2.0
>
>         Attachments: GIRAPH-192.patch, GIRAPH-192.patch, GIRAPH-192.patch, 
> GIRAPH-192.patch
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Since aggregators will be re-used throughout many projects and algorithms, it 
> makes sense to implement the most common ones in a separate sub-package. This 
> will reduce the time required for users when they implement their projects 
> based on Giraph, because the required aggregators are already in place. I 
> implemented the following ones:
> for int/long/float/double: min, max, product, sum, overwrite
> for boolean: and, or, overwrite
> Most of them speak for themselves, except for the overwrite one. This 
> aggregator simply overwrites the stored value when a new value is aggregated. 
> This is useful when one node is in some way a master node (for example a 
> source node in an routing algorithm), and this node wants to broadcast a 
> value to all other nodes.
> Attached is a patch against trunk implementing the aggregators and patching 
> some existing files so they use the .aggregators package instead of the 
> .examples one.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to