Bill Graham created PIG-2871:
--------------------------------

             Summary: Refactor signature for PigReducerEstimator
                 Key: PIG-2871
                 URL: https://issues.apache.org/jira/browse/PIG-2871
             Project: Pig
          Issue Type: Bug
            Reporter: Bill Graham
            Assignee: Bill Graham


The signature to this method should be refactored to provide more context so 
the implementer has access to the map and reduce plans. Also passing both 
{{Job}} and {{Configuration}} is redundant, since the latter can be obtained by 
the former.

I propose changing this:
{noformat}
int estimateNumberOfReducers(Configuration conf, List<POLoad> lds, Job job) 
throws IOException;
{noformat}

To this:
{noformat}
int estimateNumberOfReducers(Job job, List<POLoad> lds, MapReduceOper 
mapReduceOper) throws IOException;
{noformat}

This interface is evolving and hasn't yet been released so this is safe.

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