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

Peter Bacsko commented on HADOOP-16683:
---------------------------------------

[~adam.antal] I think the return value of this function should be {{boolean}} 
instead of {{Throwable}}, because you don't really do much with the returned 
object:

{{private static Throwable getWrappedAccessControlException(Exception e)}}
  

> Disable retry of FailoverOnNetworkExceptionRetry in case of wrapped 
> AccessControlException
> ------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-16683
>                 URL: https://issues.apache.org/jira/browse/HADOOP-16683
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: common
>    Affects Versions: 3.3.0
>            Reporter: Adam Antal
>            Assignee: Adam Antal
>            Priority: Major
>         Attachments: HADOOP-16683.001.patch
>
>
> Follow up patch on HADOOP-16580.
> We successfully disabled the retry in case of an AccessControlException which 
> has resolved some of the cases, but in other cases AccessControlException is 
> wrapped inside another IOException and you can only get the original 
> exception by calling getCause().
> Let's add this extra case as well.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to