You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by rk...@apache.org on 2018/10/11 22:35:47 UTC
hadoop git commit: HADOOP-15717. TGT renewal thread does not log
IOException (snemeth via rkanter)
Repository: hadoop
Updated Branches:
refs/heads/trunk 64f2b32d5 -> d91d47bc7
HADOOP-15717. TGT renewal thread does not log IOException (snemeth via rkanter)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/d91d47bc
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/d91d47bc
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/d91d47bc
Branch: refs/heads/trunk
Commit: d91d47bc739f23ca22a7e44fc83d449db57ab130
Parents: 64f2b32
Author: Robert Kanter <rk...@apache.org>
Authored: Thu Oct 11 15:34:41 2018 -0700
Committer: Robert Kanter <rk...@apache.org>
Committed: Thu Oct 11 15:35:44 2018 -0700
----------------------------------------------------------------------
.../hadoop/security/UserGroupInformation.java | 18 ++++++++++--------
1 file changed, 10 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/d91d47bc/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java
index db88106..915d6df 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java
@@ -920,8 +920,8 @@ public class UserGroupInformation {
final long now = Time.now();
if (tgt.isDestroyed()) {
- LOG.error("TGT is destroyed. Aborting renew thread for {}.",
- getUserName());
+ LOG.error(String.format("TGT is destroyed. " +
+ "Aborting renew thread for %s.", getUserName()), ie);
return;
}
@@ -933,16 +933,18 @@ public class UserGroupInformation {
try {
tgtEndTime = tgt.getEndTime().getTime();
} catch (NullPointerException npe) {
- LOG.error("NPE thrown while getting KerberosTicket endTime. "
- + "Aborting renew thread for {}.", getUserName());
+ LOG.error(String.format("NPE thrown while getting " +
+ "KerberosTicket endTime. Aborting renew thread for %s.",
+ getUserName()), ie);
return;
}
- LOG.warn("Exception encountered while running the renewal "
- + "command for {}. (TGT end time:{}, renewalFailures: {},"
- + "renewalFailuresTotal: {})", getUserName(), tgtEndTime,
+ LOG.warn(String.format("Exception encountered while running the " +
+ "renewal command for %s. " +
+ "(TGT end time:%d, renewalFailures: %d, " +
+ "renewalFailuresTotal: %d)", getUserName(), tgtEndTime,
metrics.renewalFailures.value(),
- metrics.renewalFailuresTotal.value(), ie);
+ metrics.renewalFailuresTotal.value()), ie);
if (rp == null) {
// Use a dummy maxRetries to create the policy. The policy will
// only be used to get next retry time with exponential back-off.
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org