You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "Daniel Dai (Updated) (JIRA)" <ji...@apache.org> on 2012/01/03 06:38:21 UTC

[jira] [Updated] (PIG-2426) ProgressableReporter.progress(String msg) is an empty function

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

Daniel Dai updated PIG-2426:
----------------------------

       Resolution: Fixed
    Fix Version/s: 0.11
     Hadoop Flags: Reviewed
           Status: Resolved  (was: Patch Available)

Unit test pass. test-patch:
     [exec] -1 overall.  
     [exec] 
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec] 
     [exec]     +1 tests included.  The patch appears to include 6 new or modified tests.
     [exec] 
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
     [exec] 
     [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
     [exec] 
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
     [exec] 
     [exec]     -1 release audit.  The applied patch generated 509 release audit warnings (more than the trunk's current 501 warnings).

All new files have proper header, ignore release audit warning.

Patch committed to 0.10/trunk.

Thanks Vivek!
                
> ProgressableReporter.progress(String msg) is an empty function
> --------------------------------------------------------------
>
>                 Key: PIG-2426
>                 URL: https://issues.apache.org/jira/browse/PIG-2426
>             Project: Pig
>          Issue Type: Bug
>    Affects Versions: 0.8.1, 0.9.1
>            Reporter: Vivek Padmanabhan
>            Assignee: Vivek Padmanabhan
>            Priority: Minor
>             Fix For: 0.10, 0.11
>
>         Attachments: PIG-2426_1.patch, PIG-2426_2.patch, TEST-org.apache.pig.test.TestAccumulator.txt, partial_run.txt
>
>
> In current implementation the reporter function ProgressableReporter.progress(String msg)  is an empty function.
> If I have a long running UDF and I want update the status using a message, the preferred way is to use this api.  
> The previous implementation of ProgressableReporter used org.apache.hadoop.mapred.Reporter api directly.
> But the currently used org.apache.hadoop.util.Progressable interface  does not have api to set status as a given message. 
> Hence I believe the empty method.

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