From 929f59978f4596f9a27604fcce72c605067d38da Mon Sep 17 00:00:00 2001 From: Lars Francke Date: Tue, 19 May 2015 21:58:09 +0200 Subject: [PATCH] HBASE-13713: See about dropping ClassLoaderBase#getClassLoadingLock --- .../apache/hadoop/hbase/util/ClassLoaderBase.java | 30 ++-------------------- 1 file changed, 2 insertions(+), 28 deletions(-) diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/util/ClassLoaderBase.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/util/ClassLoaderBase.java index 262864a..6e5aaa6 100644 --- a/hbase-common/src/main/java/org/apache/hadoop/hbase/util/ClassLoaderBase.java +++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/util/ClassLoaderBase.java @@ -19,26 +19,16 @@ package org.apache.hadoop.hbase.util; import java.net.URL; import java.net.URLClassLoader; -import java.util.concurrent.ConcurrentHashMap; - -import org.apache.hadoop.hbase.classification.InterfaceAudience; import com.google.common.base.Preconditions; +import org.apache.hadoop.hbase.classification.InterfaceAudience; /** - * Base class loader that defines couple shared constants used - * by sub-classes. It also defined method getClassLoadingLock for parallel - * class loading and JDK 1.6 support. This method (getClassLoadingLock) - * is similar to the same method in the base class Java ClassLoader - * introduced in JDK 1.7, but not in JDK 1.6. + * Base class loader that defines couple shared constants used by sub-classes. */ @InterfaceAudience.Private public class ClassLoaderBase extends URLClassLoader { - // Maps class name to the corresponding lock object - private final ConcurrentHashMap parallelLockMap - = new ConcurrentHashMap(); - protected static final String DEFAULT_LOCAL_DIR = "/tmp/hbase-local-dir"; protected static final String LOCAL_DIR_KEY = "hbase.local.dir"; @@ -59,20 +49,4 @@ public class ClassLoaderBase extends URLClassLoader { this.parent = parent; } - /** - * Returns the lock object for class loading operations. - */ - protected Object getClassLoadingLock(String className) { - Object lock = parallelLockMap.get(className); - if (lock != null) { - return lock; - } - - Object newLock = new Object(); - lock = parallelLockMap.putIfAbsent(className, newLock); - if (lock == null) { - lock = newLock; - } - return lock; - } } -- 2.4.1