Merge branch 'stable-2.16'
* stable-2.16:
FileBasedWebSessionCacheCleaner: Make CleanupTask a static inner class
Upgrade bazlets to latest stable-2.14 to build with 2.14.20 API
Change-Id: Idad0ed27096670a3a4dd5c820d4704aafdb159da
diff --git a/src/main/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleaner.java b/src/main/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleaner.java
index 0b4ada9..afc6568 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleaner.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleaner.java
@@ -35,6 +35,30 @@
private final long cleanupIntervalMillis;
private ScheduledFuture<?> scheduledCleanupTask;
+ static class CleanupTask implements Runnable {
+ private static final FluentLogger log = FluentLogger.forEnclosingClass();
+ private final FileBasedWebsessionCache fileBasedWebSessionCache;
+ private final String pluginName;
+
+ @Inject
+ CleanupTask(FileBasedWebsessionCache fileBasedWebSessionCache, @PluginName String pluginName) {
+ this.fileBasedWebSessionCache = fileBasedWebSessionCache;
+ this.pluginName = pluginName;
+ }
+
+ @Override
+ public void run() {
+ log.atInfo().log("Cleaning up expired file based websessions...");
+ fileBasedWebSessionCache.cleanUp();
+ log.atInfo().log("Cleaning up expired file based websessions...Done");
+ }
+
+ @Override
+ public String toString() {
+ return String.format("[%s] Clean up expired file based websessions", pluginName);
+ }
+ }
+
@Inject
FileBasedWebSessionCacheCleaner(
WorkQueue queue, Provider<CleanupTask> cleanupTaskProvider, Configuration config) {
@@ -63,27 +87,3 @@
}
}
}
-
-class CleanupTask implements Runnable {
- private static final FluentLogger log = FluentLogger.forEnclosingClass();
- private final FileBasedWebsessionCache fileBasedWebSessionCache;
- private final String pluginName;
-
- @Inject
- CleanupTask(FileBasedWebsessionCache fileBasedWebSessionCache, @PluginName String pluginName) {
- this.fileBasedWebSessionCache = fileBasedWebSessionCache;
- this.pluginName = pluginName;
- }
-
- @Override
- public void run() {
- log.atInfo().log("Cleaning up expired file based websessions...");
- fileBasedWebSessionCache.cleanUp();
- log.atInfo().log("Cleaning up expired file based websessions...Done");
- }
-
- @Override
- public String toString() {
- return String.format("[%s] Clean up expired file based websessions", pluginName);
- }
-}
diff --git a/src/test/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleanerTest.java b/src/test/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleanerTest.java
index 3a2828e..0a8fc92 100644
--- a/src/test/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleanerTest.java
+++ b/src/test/java/com/ericsson/gerrit/plugins/highavailability/websession/file/FileBasedWebSessionCacheCleanerTest.java
@@ -25,6 +25,7 @@
import static org.mockito.Mockito.when;
import com.ericsson.gerrit.plugins.highavailability.Configuration;
+import com.ericsson.gerrit.plugins.highavailability.websession.file.FileBasedWebSessionCacheCleaner.CleanupTask;
import com.google.gerrit.server.git.WorkQueue;
import com.google.inject.Provider;
import java.util.concurrent.ScheduledFuture;