hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "zhihai xu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-3745) SerializedException should also try to instantiate internal exception with the default constructor
Date Thu, 04 Jun 2015 22:43:38 GMT

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

zhihai xu commented on YARN-3745:
---------------------------------

[~lavkesh], thanks for working on this issue. This looks like a good catch.
One question about the patch, why retrying on SecurityException? Will retrying on NoSuchMethodException
be enough?
If need retrying on SecurityException, Can we add a test case against it?
There is a typo in the comment {{This does not has constructor with String argument}}, should
be {{have}} instead of {{has}}.
Also could we make the comment {{Try with String constructor if it fails try with default.}}
clearer as
{{Try constructor with String argument, if it fails, try default.}}
Can we add some comment to explain why ClassNotFoundException is expected in the test?


> SerializedException should also try to instantiate internal exception with the default
constructor
> --------------------------------------------------------------------------------------------------
>
>                 Key: YARN-3745
>                 URL: https://issues.apache.org/jira/browse/YARN-3745
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.6.0
>            Reporter: Lavkesh Lahngir
>            Assignee: Lavkesh Lahngir
>         Attachments: YARN-3745.1.patch, YARN-3745.patch
>
>
> While deserialising a SerializedException it tries to create internal exception in instantiateException()
with cn = cls.getConstructor(String.class).
> if cls does not has a constructor with String parameter it throws Nosuchmethodexception
> for example ClosedChannelException class.  
> We should also try to instantiate exception with default constructor so that inner exception
can to propagated.



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

Mime
View raw message