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

Gabriel Reid commented on CRUNCH-74:
------------------------------------

FWIW, I like the idea of having an initialize method as opposed to passing a 
configuration to the reset method -- this approach won't break existing code, 
and I think that anything that needs to be pulled out of the config can be done 
at initialize time.
                
> 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

Reply via email to