giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GIRAPH-725) AggregatorWrapper instantiates aggregator bypassing ReflectionUtils
Date Tue, 30 Jul 2013 19:43:50 GMT

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

Hudson commented on GIRAPH-725:
-------------------------------

FAILURE: Integrated in Giraph-trunk-Commit #1213 (See [https://builds.apache.org/job/Giraph-trunk-Commit/1213/])
GIRAPH-725: AggregatorWrapper instantiates aggregator bypassing ReflectionUtils (apresta)
(alessandro: http://git-wip-us.apache.org/repos/asf?p=giraph.git&a=commit&h=8df1027b4403f3b64ef02bd066d864371f6613f3)
* giraph-core/src/main/java/org/apache/giraph/aggregators/AggregatorWrapper.java
* giraph-core/src/main/java/org/apache/giraph/master/MasterAggregatorHandler.java

                
> AggregatorWrapper instantiates aggregator bypassing ReflectionUtils
> -------------------------------------------------------------------
>
>                 Key: GIRAPH-725
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-725
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Alessandro Presta
>            Assignee: Alessandro Presta
>         Attachments: GIRAPH-725.patch, GIRAPH-725.patch
>
>
> AggregatorWrapper creates the current aggregator by directly calling newInstance() on
its class. If the user defined a configurable aggregator, setConf() won't be called here as
one would expect, which leads to bugs.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message