summaryrefslogtreecommitdiffstats
path: root/container-accesslogging/src/main/java/com/yahoo/container/logging
diff options
context:
space:
mode:
Diffstat (limited to 'container-accesslogging/src/main/java/com/yahoo/container/logging')
-rw-r--r--container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogEntry.java22
-rw-r--r--container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogHandler.java8
-rw-r--r--container-accesslogging/src/main/java/com/yahoo/container/logging/LogFileHandler.java19
3 files changed, 6 insertions, 43 deletions
diff --git a/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogEntry.java b/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogEntry.java
index 452466d7d05..46e5aa95dab 100644
--- a/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogEntry.java
+++ b/container-accesslogging/src/main/java/com/yahoo/container/logging/AccessLogEntry.java
@@ -61,7 +61,6 @@ public class AccessLogEntry {
private long timeStampMillis;
private long durationBetweenRequestResponseMillis;
private long numBytesReturned;
- private URI uri;
private String remoteAddress;
@@ -600,27 +599,6 @@ public class AccessLogEntry {
}
}
- /**
- * @deprecated Use {@link #setRawPath(String)} and {@link #setRawQuery(String)} instead.
- */
- @Deprecated
- public void setURI(final URI uri) {
- synchronized (monitor) {
- requireNull(this.uri);
- this.uri = uri;
- }
- }
-
- /**
- * @deprecated Use {@link #getRawPath()} and {@link #getRawQuery()} instead. This method may return wrong path.
- */
- @Deprecated
- public URI getURI() {
- synchronized (monitor) {
- return uri;
- }
- }
-
public void setRemoteAddress(String remoteAddress) {
synchronized (monitor) {
requireNull(this.remoteAddress);
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..3912a3be853 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,17 +99,12 @@ 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;
logThread = new LogThread(this);
logThread.start();
@@ -237,22 +230,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;