Track removal of fileHandler.rotateSchema from access-log.def

3 files changed, 11 insertions, 43 deletions

diff --git a/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogHandler.java b/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogHandler.java
index a31e7b3b849..488a6137cc2 100644
--- a/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogHandler.java
+++ b/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogHandler.java
@@ -3,8 +3,6 @@ package com.yahoo.container.logging;
 
 import com.yahoo.container.core.AccessLogConfig;
 
-import static com.yahoo.container.core.AccessLogConfig.FileHandler.RotateScheme.DATE;
-
 import java.util.logging.Logger;
 
 /**
@@ -18,14 +16,12 @@ class AccessLogHandler {
     public AccessLogHandler(AccessLogConfig.FileHandler config) {
         access.setUseParentHandlers(false);
 
-        logFileHandler = new LogFileHandler(config.rotateScheme(), config.compressOnRotation());
+        logFileHandler = new LogFileHandler(config.compressOnRotation());
 
         logFileHandler.setFilePattern(config.pattern());
         logFileHandler.setRotationTimes(config.rotation());
 
-        if (config.rotateScheme() == DATE)
-            createSymlink(config, logFileHandler);
-
+        createSymlink(config, logFileHandler);
 
         LogFormatter lf = new LogFormatter();
         lf.messageOnly(true);
diff --git a/container-accesslogging/src/main/java/com/yahoo/container/logging/LogFileHandler.java b/container-accesslogging/src/main/java/com/yahoo/container/logging/LogFileHandler.java
index 1f56523bce6..135c3cd7af0 100644
--- a/container-accesslogging/src/main/java/com/yahoo/container/logging/LogFileHandler.java
+++ b/container-accesslogging/src/main/java/com/yahoo/container/logging/LogFileHandler.java
@@ -38,8 +38,6 @@ import java.util.zip.GZIPOutputStream;
 public class LogFileHandler extends StreamHandler {
 
     private final static Logger logger = Logger.getLogger(LogFileHandler.class.getName());
-    /** True to use the sequence file name scheme, false (default) to use the date scheme */
-    private final boolean useSequenceNameScheme;
     private final boolean compressOnRotation;
     private long[] rotationTimes = {0}; //default to one log per day, at midnight
     private String filePattern = "./log.%T";  // default to current directory, ms time stamp
@@ -101,18 +99,17 @@ public class LogFileHandler extends StreamHandler {
     private final LogThread logThread;
 
     LogFileHandler() {
-        this(AccessLogConfig.FileHandler.RotateScheme.Enum.DATE, false);
+        this(false);
     }
 
-    LogFileHandler(boolean compressOnRotation) {
-        this(AccessLogConfig.FileHandler.RotateScheme.Enum.DATE, compressOnRotation);
-    }
-
-    LogFileHandler(AccessLogConfig.FileHandler.RotateScheme.Enum rotateScheme, boolean compressOnRotation)
+    LogFileHandler(boolean compressOnRotation)
     {
         super();
-        this.useSequenceNameScheme = (rotateScheme == AccessLogConfig.FileHandler.RotateScheme.Enum.SEQUENCE);
         this.compressOnRotation = compressOnRotation;
+        init();
+    }
+    
+    private void init() {
         logThread = new LogThread(this);
         logThread.start();
     }
@@ -237,22 +234,18 @@ public class LogFileHandler extends StreamHandler {
         super.flush();
         super.close();
 
-        if (useSequenceNameScheme)
-            moveCurrentFile();
-
         try {
             checkAndCreateDir(fileName);
             FileOutputStream os = new FileOutputStream(fileName, true); // append mode, for safety
             super.setOutputStream(os);
             currentOutputStream = os;
-            if (! useSequenceNameScheme) LogFileDb.nowLoggingTo(fileName);
+            LogFileDb.nowLoggingTo(fileName);
         }
         catch (IOException e) {
             throw new RuntimeException("Couldn't open log file '" + fileName + "'", e);
         }
 
-        if ( ! useSequenceNameScheme)
-            createSymlinkToCurrentFile();
+        createSymlinkToCurrentFile();
 
         numberOfRecords = 0;
         lastRotationTime = now;
diff --git a/container-accesslogging/src/test/java/com/yahoo/container/logging/YApacheLogTestCase.java b/container-accesslogging/src/test/java/com/yahoo/container/logging/YApacheLogTestCase.java
index a643ba77703..30442089f18 100644
--- a/container-accesslogging/src/test/java/com/yahoo/container/logging/YApacheLogTestCase.java
+++ b/container-accesslogging/src/test/java/com/yahoo/container/logging/YApacheLogTestCase.java
@@ -187,26 +187,6 @@ public class YApacheLogTestCase {
         return entry;
     }
 
-    /**
-     * author someone-else. Please rewrite this.
-     */
-    @Test
-    public void testYApacheAccessLogWithSequenceNamingScheme() throws IOException, InterruptedException {
-        // try without existing files
-        assertCorrectSequenceBehavior(1);
-
-        // try with existing files
-        try {
-            new File("yapachetest2").mkdir();
-            new File("yapachetest2/access.1").createNewFile();
-            new File("yapachetest2/access.2").createNewFile();
-            assertCorrectSequenceBehavior(3);
-        }
-        finally {
-            deleteDirectory("yapachetest2");
-        }
-    }
-
     // Prefixes that don't collide with any in the specified log format.
     private static final char FIELD_KEY_REQUEST_EXTRA = '@';
     private static final char FIELD_KEY_RESPONSE_EXTRA = '#';
@@ -232,8 +212,7 @@ public class YApacheLogTestCase {
         AccessLogConfig.Builder builder = new AccessLogConfig.Builder().
                 fileHandler(new AccessLogConfig.FileHandler.Builder().
                         pattern("yapachetest2/access").
-                        compressOnRotation(false).
-                        rotateScheme(AccessLogConfig.FileHandler.RotateScheme.Enum.SEQUENCE));
+                        compressOnRotation(false));
 
         AccessLogConfig config = new AccessLogConfig(builder);
         YApacheAccessLog accessLog = new YApacheAccessLog(config);