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

Hitesh Shah commented on TEZ-2292:
----------------------------------

For this patch, minor comment: 

{code}
} catch (IOException e) {
        e.printStackTrace();
 }
{code}
   - maybe re-throw this back as a runtime exception instead of ignoring it? 

+1 once the above is fixed. 




> 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