[ 
https://issues.apache.org/jira/browse/PIG-2574?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bill Graham updated PIG-2574:
-----------------------------

    Attachment: pig-2574_4.patch

Here's a patch with Dmitriy's comments, plus a few other small edits:

* Incorporated the fix from PIG-2616 into {{InputSizeReducerEstimator}}.
* Added both {{@InterfaceAudience.Public}} and {{@InterfaceStability.Evolving}} 
to {{PigReducerEstimator}}.
* Moved some tests from {{TestJobControlCompiler}} to 
{{TestInputSizeReducerEstimator}}. 

Also, I added the following to {{.gitignore}}. Let me know if I should back 
that out and put it in it's own patch:

{noformat}
+target/
+*.iml
+*.ipr
+*.iws
{noformat}

                
> Make reducer estimator plugable
> -------------------------------
>
>                 Key: PIG-2574
>                 URL: https://issues.apache.org/jira/browse/PIG-2574
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>              Labels: 0.10_blocker
>             Fix For: 0.10, 0.11
>
>         Attachments: pig-2574_1.patch, pig-2574_2.patch, pig-2574_3.patch, 
> pig-2574_4.patch
>
>
> I'd like to refactor the logic contained in this method into a pluggable 
> interface:
> {noformat}
> static int JobControlCompiler.estimateNumberOfReducers(Configuration conf, 
> List<POLoad> lds);
> {noformat}

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