[jira] [Comment Edited] (TEZ-2292) Add e2e test for error reporting when vertex manager invokes plugin APIs

["Hitesh Shah (JIRA)" <ji...@apache.org>]

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