Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HADOOP-14310. RolloverSignerSecretProvider.LOG should be @VisibleForTesting 

(Contributed by Arun Shanmugam Kumar via Daniel Templeton)
This commit is contained in:
Daniel Templeton 2017-06-12 09:42:16 -07:00
parent d64c842743
commit 86368cc766
2 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hadoop-common-project/hadoop-auth/pom.xml
View File

@ -181,6 +181,11 @@
<version>${apacheds.version}</version> <version>${apacheds.version}</version>
<scope>test</scope> <scope>test</scope>
</dependency> </dependency>
<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
<scope>compile</scope>
</dependency>
</dependencies> </dependencies>
<build> <build>

2
hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/util/RolloverSignerSecretProvider.java
View File

@ -18,6 +18,7 @@ import java.util.concurrent.Executors;
import java.util.concurrent.ScheduledExecutorService; import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.TimeUnit; import java.util.concurrent.TimeUnit;
import javax.servlet.ServletContext; import javax.servlet.ServletContext;
import com.google.common.annotations.VisibleForTesting;
import org.apache.hadoop.classification.InterfaceAudience; import org.apache.hadoop.classification.InterfaceAudience;
import org.apache.hadoop.classification.InterfaceStability; import org.apache.hadoop.classification.InterfaceStability;
import org.slf4j.Logger; import org.slf4j.Logger;
@ -38,6 +39,7 @@ import org.slf4j.LoggerFactory;
public abstract class RolloverSignerSecretProvider public abstract class RolloverSignerSecretProvider
extends SignerSecretProvider { extends SignerSecretProvider {
@VisibleForTesting
static Logger LOG = LoggerFactory.getLogger( static Logger LOG = LoggerFactory.getLogger(
RolloverSignerSecretProvider.class); RolloverSignerSecretProvider.class);
/** /**