由于log4j配置,打开的文件太多

时间:2017-11-07 19:21:11

标签: java file log4j2

我们正在使用log4j 2.5。我的问题是关于 ConferenceLog 所以请忽略下面的其他配置。

log4j2.xml文件:

<?xml version="1.0" encoding="UTF-8"?>
    <Configuration monitorInterval="180">
        <Properties>
            <!-- change the path to conference logs below to fit your system -->
            <Property name="LOG_DIR">C:\CodeRepos\logs</Property>
            <Property name="PATTERN">%d{yyyy-MM-dd HH:mm:ss.SSS} [%t] %-5level
                %logger{36} - %msg%n</Property>
        </Properties>
        <Appenders>
            <Console name="Console" target="SYSTEM_OUT">
                <PatternLayout pattern="${PATTERN}" />
            </Console>
            <Routing name="DailyLog">
                <Routes pattern="${ctx:logFileName}">
                    <Route>
                        <RollingFile name="Rolling-${ctx:logFileName}"
                            fileName="${LOG_DIR}/symproxy.log" 
                            filePattern="${LOG_DIR}/symproxy-%i.log">
                            <PatternLayout pattern="${PATTERN}" />
                            <SizeBasedTriggeringPolicy size="20971520" />
                            <DefaultRolloverStrategy max="100" />
                        </RollingFile>
                    </Route>
                </Routes>
            </Routing>
            <Routing name="ConferenceLog">
                <Routes pattern="${ctx:logFileName}">
                    <Route>
                        <RollingFile name="Rolling-${ctx:logFileName}"
                            fileName="${LOG_DIR}/conferences/${ctx:logFileName}.log"
                            filePattern="${LOG_DIR}/conferences/${ctx:logFileName}.%i.log.gz">
                            <PatternLayout pattern="${PATTERN}" />
                            <SizeBasedTriggeringPolicy size="20971520" />
                        </RollingFile>
                    </Route>
                </Routes>
            </Routing>
            <Routing name="DeviceLog">
                <Routes pattern="${ctx:logFileName}">
                    <Route>
                        <RollingFile name="Rolling-${ctx:logFileName}"
                            fileName="${LOG_DIR}/devices/${ctx:logFileName}.log"
                            filePattern="${LOG_DIR}/devices/${ctx:logFileName}.%i.log.gz">
                            <PatternLayout pattern="${PATTERN}" />
                            <SizeBasedTriggeringPolicy size="5kb" />
                            <DefaultRolloverStrategy max="3" />
                        </RollingFile>
                    </Route>
                </Routes>
            </Routing>
        </Appenders>
        <Loggers>
            <Root level="info">
                <AppenderRef ref="Console" />
                <AppenderRef ref="DailyLog" />
            </Root>
            <Logger name="conference.logger" level="debug" additivity="false">
                <AppenderRef ref="ConferenceLog" />
                <AppenderRef ref="Console" />
            </Logger>
            <Logger name="device.logger" level="info" additivity="false">
            </Logger>
        </Loggers>
    </Configuration>

我有一个小包装器,在那里我实例化这个会议记录器

private final Log conferenceLogger = LogFactory.getLog("conference.logger");

之后就像这样使用

private void logger(Long meetingId, Log log, String className, Level level, String message, Throwable error) {
        ThreadContext.put(LOG_FILE_NAME, String.valueOf(meetingId));
        logger(conferenceLogger, level, className + "." + message, error);
        ThreadContext.remove(LOG_FILE_NAME);
    }

    private static void logger(Log log, Level level, String message, Throwable error) {
        if (level == Level.DEBUG) {
            log.debug(message);
        } else if (level == Level.ERROR) {
            if (error == null) {
                log.error(message);
            } else {
                log.error(message, error);
            }
        } else if (level == Level.FATAL) {
            log.fatal(message);
        } else if (level == Level.TRACE) {
            log.trace(message);
        } else if (level == Level.INFO) {
            log.info(message);
        } else if (level == Level.WARN) {
            log.warn(message);
        }
    }

这会创建一组以“会议ID”命名的文件。然而,我的问题是这些文件在打开后永远不会关闭。

我已阅读此similar SO post但是我没有引用我的FileAppender来调用close,也没有Logger对象,所以我不确定实施其修复。如何以编程方式触发这些文件的关闭? 我们的一台服务器因此而收到“太多打开的文件”错误。

阅读documentation之后,似乎IdlePurgePolicy应该是我需要的,但我不认为它对打开文件的数量有任何影响。 我尝试下面的行,如果我的log4j2文件的一些不同的地方,但正如我提到的,它没有影响

<IdlePurgePolicy timeToLive="1" timeUnit="minutes"/>

2 个答案:

答案 0 :(得分:0)

如果包装器本身没有静态实例化,则将为每个调用它的对象创建记录器实例。这可能导致每次都与具体记录器实例建立新的套接字连接。但是,如果只有一个日志记录文件,过多的打开套接字也可能导致此异常。

可能的解决方案

  1. 静态实例化包装器
  2. 静态实例化记录器参考
  3. 使用日志调用日志记录方法
  4. 使包装器成为单例

答案 1 :(得分:0)

IdlePurgePolicy最终确定了这个

<Routing name="ConferenceLog">
    <Routes pattern="${ctx:logFileName}">
        <Route>
            <RollingFile name="Rolling-${ctx:logFileName}"
                fileName="${LOG_DIR}/conferences/${ctx:logFileName}.log"
                filePattern="${LOG_DIR}/conferences/${ctx:logFileName}.%i.log.gz">
                <PatternLayout pattern="${PATTERN}" />
                <SizeBasedTriggeringPolicy size="20971520" />
            </RollingFile>
        </Route>
    </Routes>
    <IdlePurgePolicy timeToLive="30" timeUnit="seconds"/>
</Routing>