You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by "robin zhang tao (JIRA)" <ji...@apache.org> on 2013/11/22 09:22:35 UTC
[jira] [Created] (LOG4J2-455) RingBufferLogEvent should use
Messsage timestamp first
robin zhang tao created LOG4J2-455:
--------------------------------------
Summary: RingBufferLogEvent should use Messsage timestamp first
Key: LOG4J2-455
URL: https://issues.apache.org/jira/browse/LOG4J2-455
Project: Log4j 2
Issue Type: Bug
Components: Appenders
Affects Versions: 2.0-beta9
Reporter: robin zhang tao
When AsyncLogger received log message (method public void log(final Marker marker, final String fqcn, final Level level, final Message data,
final Throwable t)), currently it uses clock.currentTimeMillis() (see at Line 221) as the timestamp.
But it should check the message implements TimestampMessage or not, if yes, should use TimestampMessage.getTimestamp() as the timestamp, if not , then using clock.currentTimeMillis().
The new Code:
data instanceof TimestampMessage ? ((TimestampMessage) data ).getTimestamp() : clock.currentTimeMillis();
--
This message was sent by Atlassian JIRA
(v6.1#6144)
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org