[ https://issues.apache.org/jira/browse/HADOOP-13590?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15511441#comment-15511441 ]
Kai Zheng commented on HADOOP-13590: ------------------------------------ Thanks for proposing and working on this, [~xiaochen]. Some quick comments: 1. The new metric looks good. Maybe: renewalFailure => renewalFailures, also the method. {code} + @Metric("Renewal failures since last successful login") + private long renewalFailure; {code} 2. The new test based on mock and faked things looks good. Is it possible to add some tests utilizing MiniKDC? For example, trying some cases like KDC being stopped then restarted. This would bring more confidence. Please feel free to open new one if you want to address this separately. > Retry until TGT expires even if the UGI renewal thread encountered exception > ---------------------------------------------------------------------------- > > Key: HADOOP-13590 > URL: https://issues.apache.org/jira/browse/HADOOP-13590 > Project: Hadoop Common > Issue Type: Improvement > Components: security > Affects Versions: 2.8.0, 2.7.3, 2.6.4 > Reporter: Xiao Chen > Assignee: Xiao Chen > Attachments: HADOOP-13590.01.patch, HADOOP-13590.02.patch, > HADOOP-13590.03.patch, HADOOP-13590.04.patch > > > The UGI has a background thread to renew the tgt. On exception, it > [terminates > itself|https://github.com/apache/hadoop/blob/bee9f57f5ca9f037ade932c6fd01b0dad47a1296/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java#L1013-L1014] > If something temporarily goes wrong that results in an IOE, even if it > recovered no renewal will be done and client will eventually fail to > authenticate. We should retry with our best effort, until tgt expires, in the > hope that the error recovers before that. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org