From 86368cc7667adfe224df77b039c2ffe8de7f889a Mon Sep 17 00:00:00 2001 From: Daniel Templeton Date: Mon, 12 Jun 2017 09:42:16 -0700 Subject: [PATCH] HADOOP-14310. RolloverSignerSecretProvider.LOG should be @VisibleForTesting (Contributed by Arun Shanmugam Kumar via Daniel Templeton) --- hadoop-common-project/hadoop-auth/pom.xml | 5 +++++ .../authentication/util/RolloverSignerSecretProvider.java | 2 ++ 2 files changed, 7 insertions(+) diff --git a/hadoop-common-project/hadoop-auth/pom.xml b/hadoop-common-project/hadoop-auth/pom.xml index f608daa675..61bc8b4742 100644 --- a/hadoop-common-project/hadoop-auth/pom.xml +++ b/hadoop-common-project/hadoop-auth/pom.xml @@ -181,6 +181,11 @@ ${apacheds.version} test + + com.google.guava + guava + compile + diff --git a/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/util/RolloverSignerSecretProvider.java b/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/util/RolloverSignerSecretProvider.java index 66b2fde71a..e516b5b76a 100644 --- a/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/util/RolloverSignerSecretProvider.java +++ b/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/util/RolloverSignerSecretProvider.java @@ -18,6 +18,7 @@ import java.util.concurrent.ScheduledExecutorService; import java.util.concurrent.TimeUnit; import javax.servlet.ServletContext; +import com.google.common.annotations.VisibleForTesting; import org.apache.hadoop.classification.InterfaceAudience; import org.apache.hadoop.classification.InterfaceStability; import org.slf4j.Logger; @@ -38,6 +39,7 @@ public abstract class RolloverSignerSecretProvider extends SignerSecretProvider { + @VisibleForTesting static Logger LOG = LoggerFactory.getLogger( RolloverSignerSecretProvider.class); /**