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

Kan Zhang commented on HADOOP-6706:
-----------------------------------

+1, patch looks good. Couple nits.

bq. It's better to call disposeSasl() immediately after closeConnection() in 
handleSaslConnectionFailure().

bq. Please don't format lines you're not changing otherwise. This will make 
patch easier to read.

> Relogin behavior for RPC clients could be improved
> --------------------------------------------------
>
>                 Key: HADOOP-6706
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6706
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: security
>    Affects Versions: 0.22.0
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: 6706-bp20-2.patch, 6706.bp20.1.patch, 6706.bp20.patch, 
> HADOOP-6706-BP20-fix1.patch, HADOOP-6706-BP20-fix2.patch, 
> HADOOP-6706-BP20-fix3.patch, HADOOP-6706.2.patch, HADOOP-6706.4.patch, 
> HADOOP-6706.5.patch
>
>
> Currently, the relogin in the RPC client happens on only a SaslException. But 
> we have seen cases where other exceptions are thrown (like 
> IllegalStateException when the client's ticket is invalid). This jira is to 
> fix that behavior.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to