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

Bill Graham updated PIG-2660:
-----------------------------

    Attachment: PIG-2660.2.patch

Cool call. I haven't heard of anyone else using PPNLs on the list yet, but I 
agree with the suggestion. Patch #2 attached.

I wondered if we should we take it a step further and catch Exception for all 
calls to a PPNL. Should a bug in a PPNL implementation cause a script to fail, 
possibly mid-way through execution?

If we leave things as they are now, the answer would be _yes_, unless someone 
implemented their PPNL defensively to make it a _no_. As such, I'm inclined to 
leave things as they are and put the control in the hands of the implementer.
                
> PPNL should get notified of plan before it gets executed
> --------------------------------------------------------
>
>                 Key: PIG-2660
>                 URL: https://issues.apache.org/jira/browse/PIG-2660
>             Project: Pig
>          Issue Type: New Feature
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>         Attachments: PIG-2660.1.patch, PIG-2660.2.patch
>
>
> The {{PigProgressNotificationListner}} should get notified of the plan 
> ({{MROperPlan}}) before it gets executed. This allows listeners to inspect 
> the plan and have an idea what to expect in the execution flow. Proposal is 
> to add the following method to the PPNL interface, which is marked as 
> evolving:
> {code}
> public void initialPlanNotification(MROperPlan plan);
> {code}

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