[
https://issues.apache.org/jira/browse/HADOOP-12897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16332879#comment-16332879
]
Arpit Agarwal commented on HADOOP-12897:
----------------------------------------
Couple of minor comments:
# We should probably wrap AuthenticationException in addition to IOException.
# You can skip this log message.
{code}
LOG.warn("Unable to wrap exception of type " +
exceptionClass + ": it has no (String) constructor", e);
{code}
# Coding style: Need spaces after try and before catch keywords.
> KerberosAuthenticator.authenticate to include URL on IO failures
> ----------------------------------------------------------------
>
> Key: HADOOP-12897
> URL: https://issues.apache.org/jira/browse/HADOOP-12897
> Project: Hadoop Common
> Issue Type: Improvement
> Components: security
> Affects Versions: 2.8.0
> Reporter: Steve Loughran
> Assignee: Ajay Kumar
> Priority: Minor
> Attachments: HADOOP-12897.001.patch, HADOOP-12897.002.patch
>
>
> If {{KerberosAuthenticator.authenticate}} can't connect to the endpoint, you
> get a stack trace, but without the URL it is trying to talk to.
> That is: it doesn't have any equivalent of the {{NetUtils.wrapException}}
> handler —which can't be called here as its not in the {{hadoop-auth}} module
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
