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

Hitesh Shah commented on TEZ-3029:
----------------------------------

bq. In terms of package access, that can be done by subclassing for the 
MockDagAppMaster - which is in a different package, and hence public

Thanks for the clarification. No need to change. 

+1 from a functional perspective ( pending the addressing of the review 
comments related to the API discussion )

> Add an onError method to service plugin contexts
> ------------------------------------------------
>
>                 Key: TEZ-3029
>                 URL: https://issues.apache.org/jira/browse/TEZ-3029
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: TEZ-3029.1.txt, TEZ-3029.2.txt, TEZ-3029.3.txt
>
>
> This is to indicate errors which may occur while running threads etc.
> One bit to be careful about is that this introduces two means of reporting 
> errors - which has been challenging to handle in the past for 
> Input/Processor/OutputContext - so whether to do this or not needs to be 
> thought through.
> TaskSchedulerContext already has this method.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to