diff options
author | Tor Egge <Tor.Egge@broadpark.no> | 2018-09-20 12:24:04 +0200 |
---|---|---|
committer | gjoranv <gv@oath.com> | 2019-01-21 15:09:22 +0100 |
commit | 7b58424ddd4cdf0c2e963cd89bac83dcf8566381 (patch) | |
tree | a0d1c4b7f8fecbc9d999ce8417e78b40bdc6f7bd /container-accesslogging | |
parent | b940326231702e8a07dc7e2d3fb5da440bf171cf (diff) |
Track removal of fileHandler.rotateSchema from access-log.def
Diffstat (limited to 'container-accesslogging')
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); |