incubator-crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Wills (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CRUNCH-74) Reset method of Agregator should have a Configuration param
Date Tue, 25 Sep 2012 23:06:09 GMT

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

Josh Wills commented on CRUNCH-74:
----------------------------------

@Gabriel agree with your comment w/the exception that we can't help but break existing code--
Aggregator is an interface, so the new initialize(Configuration) method will need to be added
everywhere.
                
> Reset method of Agregator should have a Configuration param
> -----------------------------------------------------------
>
>                 Key: CRUNCH-74
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-74
>             Project: Crunch
>          Issue Type: New Feature
>            Reporter: Rahul Sharma
>            Priority: Minor
>         Attachments: 0001-Adding-configuration-to-reset-method-in-Aggregator.patch
>
>
> The rest method of aggregator is called for every key that is being received at the reducer.
The purpose of the reset method is to re-initialize any values if required. The initialization
may require some job params or user params from configuration object. There should be a parameter
to reset method which will have the configuration that is being used.

--
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