honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix example in logging daily rotate configuration 

PR #8464 come with a bug in the example provided.

First, the current log file is not compressed so it should not end with `.gz`.
Second, conversion pattern was removing all the log content but was printing only the log date.
Then, the current log filename was hardcoded to `elasticsearch` instead of the cluster name.
This commit is contained in:
David Pilato 2014-11-19 15:49:37 +01:00
parent c297ca1668
commit cbced948c4
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
config/logging.yml
View File

@ -43,12 +43,12 @@ appender:
# For more information see https://logging.apache.org/log4j/extras/apidocs/org/apache/log4j/rolling/RollingFileAppender.html # For more information see https://logging.apache.org/log4j/extras/apidocs/org/apache/log4j/rolling/RollingFileAppender.html
#file: #file:
#type: extrasRollingFile #type: extrasRollingFile
#file: ${path.logs}/elasticsearch.log.gz #file: ${path.logs}/${cluster.name}.log
#rollingPolicy: timeBased #rollingPolicy: timeBased
#rollingPolicy.FileNamePattern: ${path.logs}/${cluster.name}%d{yyyy-MM-dd}.log.gz #rollingPolicy.FileNamePattern: ${path.logs}/${cluster.name}.log.%d{yyyy-MM-dd}.gz
#layout: #layout:
#type: pattern #type: pattern
#conversionPattern: "%d{ISO8601}" #conversionPattern: "[%d{ISO8601}][%-5p][%-25c] %m%n"
index_search_slow_log_file: index_search_slow_log_file:
type: dailyRollingFile type: dailyRollingFile

3
src/main/java/org/elasticsearch/common/logging/log4j/LogConfigurator.java
View File

@ -62,12 +62,13 @@ public class LogConfigurator {
.put("null", "org.apache.log4j.NullAppender") .put("null", "org.apache.log4j.NullAppender")
.put("rollingFile", "org.apache.log4j.RollingFileAppender") .put("rollingFile", "org.apache.log4j.RollingFileAppender")
.put("extrasRollingFile", "org.apache.log4j.rolling.RollingFileAppender") .put("extrasRollingFile", "org.apache.log4j.rolling.RollingFileAppender")
.put("timeBased", "org.apache.log4j.rolling.TimeBasedRollingPolicy")
.put("smtp", "org.apache.log4j.net.SMTPAppender") .put("smtp", "org.apache.log4j.net.SMTPAppender")
.put("socket", "org.apache.log4j.net.SocketAppender") .put("socket", "org.apache.log4j.net.SocketAppender")
.put("socketHub", "org.apache.log4j.net.SocketHubAppender") .put("socketHub", "org.apache.log4j.net.SocketHubAppender")
.put("syslog", "org.apache.log4j.net.SyslogAppender") .put("syslog", "org.apache.log4j.net.SyslogAppender")
.put("telnet", "org.apache.log4j.net.TelnetAppender") .put("telnet", "org.apache.log4j.net.TelnetAppender")
// policies
.put("timeBased", "org.apache.log4j.rolling.TimeBasedRollingPolicy")
// layouts // layouts
.put("simple", "org.apache.log4j.SimpleLayout") .put("simple", "org.apache.log4j.SimpleLayout")
.put("html", "org.apache.log4j.HTMLLayout") .put("html", "org.apache.log4j.HTMLLayout")