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 jl...@apache.org on 2014/02/18 16:23:30 UTC
svn commit: r1569382 - in
/hadoop/common/trunk/hadoop-common-project/hadoop-common: CHANGES.txt
src/main/java/org/apache/hadoop/security/token/Token.java
Author: jlowe
Date: Tue Feb 18 15:23:29 2014
New Revision: 1569382
URL: http://svn.apache.org/r1569382
Log:
HADOOP-10346. Deadlock while logging tokens. Contributed by Jason Lowe
Modified:
hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/Token.java
Modified: hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt?rev=1569382&r1=1569381&r2=1569382&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt (original)
+++ hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt Tue Feb 18 15:23:29 2014
@@ -357,6 +357,8 @@ Release 2.4.0 - UNRELEASED
HADOOP-10249. LdapGroupsMapping should trim ldap password read from file.
(Dilli Armugam via suresh)
+ HADOOP-10346. Deadlock while logging tokens (jlowe)
+
Release 2.3.1 - UNRELEASED
INCOMPATIBLE CHANGES
Modified: hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/Token.java
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/Token.java?rev=1569382&r1=1569381&r2=1569382&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/Token.java (original)
+++ hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/Token.java Tue Feb 18 15:23:29 2014
@@ -105,18 +105,21 @@ public class Token<T extends TokenIdenti
return identifier;
}
- private static synchronized Class<? extends TokenIdentifier>
+ private static Class<? extends TokenIdentifier>
getClassForIdentifier(Text kind) {
- if (tokenKindMap == null) {
- tokenKindMap = Maps.newHashMap();
- for (TokenIdentifier id : ServiceLoader.load(TokenIdentifier.class)) {
- tokenKindMap.put(id.getKind(), id.getClass());
+ Class<? extends TokenIdentifier> cls = null;
+ synchronized (Token.class) {
+ if (tokenKindMap == null) {
+ tokenKindMap = Maps.newHashMap();
+ for (TokenIdentifier id : ServiceLoader.load(TokenIdentifier.class)) {
+ tokenKindMap.put(id.getKind(), id.getClass());
+ }
}
+ cls = tokenKindMap.get(kind);
}
- Class<? extends TokenIdentifier> cls = tokenKindMap.get(kind);
if (cls == null) {
LOG.warn("Cannot find class for token kind " + kind);
- return null;
+ return null;
}
return cls;
}