[jira] [Commented] (CRUNCH-74) Reset method of Agregator should have a Configuration param

["Josh Wills (JIRA)" <ji...@apache.org>]

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

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

I see the need for something like this, but walk me through the pros-and-cons of chaning the reset method signature vs. adding an initialize(Configuration conf) method to the Aggregator interface.
                
> 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