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

Hitesh Shah edited comment on TEZ-2292 at 4/21/15 6:14 PM:
-----------------------------------------------------------

If a user plugin is invoked, we should be catching Exception in any case and 
not just TezException unless the framework layer is re-wrapping the user code 
exception. 

In this scenario, it is the framework code that is throwing an exception back 
to the user-code. In that scenario, we should be declaring  a throws exception 
on the re-configure api ( though adding a new exception to an existing api is 
incompatible except for certain cases ). 


was (Author: hitesh):
If a user plugin is invoked, we should be catching Exception in any case and 
not just TezException unless the framework layer is re-wrapping the user code 
exception. 

> Add e2e test for error reporting when vertex manager invokes plugin APIs
> ------------------------------------------------------------------------
>
>                 Key: TEZ-2292
>                 URL: https://issues.apache.org/jira/browse/TEZ-2292
>             Project: Apache Tez
>          Issue Type: Task
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>            Priority: Blocker
>             Fix For: 0.7.0
>
>         Attachments: TEZ-2292.1.patch
>
>
> If the Vertex Manager has an error or cannot apply a required reconfiguration 
> then it should be allowed to fail the vertex.



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

Reply via email to