[
https://issues.apache.org/jira/browse/TEZ-2745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14727087#comment-14727087
]
Jeff Zhang commented on TEZ-2745:
---------------------------------
Upload new patch.
bq. from a quick look at the patch, I dont see any public apis needing to
change to add a "throws TezException" - can you confirm?
No public api in tez-api is changed.
* Create new exception class TezReflectionException
* The exception from constructor is caught in ReflectionUtils. Because the user
constructor is not called explicitly. it is invoked by reflection.
* 2 places is changed to use ReflectionUtils. (ContainerLauncherManager &
TaskSchedulerManager). The only difference between using ReflectionUtils and
the original way is using ReflectionUtils require the constructor to be public.
That's why I make some changes in TestContainerLauncherManager &
TestTaskSchedulerManager
> ClassNotFoundException of user code should fail dag
> ---------------------------------------------------
>
> Key: TEZ-2745
> URL: https://issues.apache.org/jira/browse/TEZ-2745
> Project: Apache Tez
> Issue Type: Bug
> Affects Versions: 0.7.0, 0.5.4, 0.6.2, 0.8.0-alpha
> Reporter: Jeff Zhang
> Assignee: Jeff Zhang
> Attachments: TEZ-2745-1.patch, TEZ-2745-2.patch, TEZ-2745-3.patch
>
>
> This ClassNotFoundException is not captured now. The current behavior is AM
> crashed and relaunched again until max app attempt is reached.
> Here's user code used in AM:
> * EdgeManager
> * VertexManager
> * InputInitializer
> * OutputCommitter
> * Other user pluggable components (like DAGScheduler, HistoryServiceLogging
> etc.)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
