[ 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