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

Bill Graham commented on PIG-2664:
----------------------------------

There's also EmbeddedPigStats which is used when Pig is running in embedded 
mode. It also subclasses PigStats but throws UnsupportedOperationException 
exceptions for most of the abstract methods it implements. We can follow the 
same approach with an abstract getJobClient() method.

I also don't love the ((SimplePigStats)PigStats.get()).getFoo() approach, but 
it's the pattern already in use in PigStatsUtil and other options didn't seem 
great. I'm ok with moving getJobClient it to PigStats though. That way we can 
at least get rid of the cast.
                
> Allow PPNL impls to get more job info during the run
> ----------------------------------------------------
>
>                 Key: PIG-2664
>                 URL: https://issues.apache.org/jira/browse/PIG-2664
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>         Attachments: PIG-2664.1.patch
>
>
> Implementations of PigProcessNotificationListener don't have access to much 
> job-related into until after the job is completed. A few minor edits to 
> PigStatsUtil can change that.
> - Expose the JobClient
> - Set an initial JobStats object upon kicking off a job. This makes JobId 
> available to clients once it's assigned.

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