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 "Gary Gregory (JIRA)" <ji...@apache.org> on 2015/06/10 05:36:01 UTC

[jira] [Commented] (LOG4J2-982) Use System.nanoTime() to measure time intervals

    [ https://issues.apache.org/jira/browse/LOG4J2-982?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14579979#comment-14579979 ] 

Gary Gregory commented on LOG4J2-982:
-------------------------------------

I'm OK with the idea of this patch based on the links in https://issues.apache.org/jira/browse/LOG4J2-982?focusedCommentId=14375141&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14375141

Would you be willing to re-create the patch for the current Git master as the it no longer applies cleanly.

I would also like to see the kind of improvement mentioned in https://issues.apache.org/jira/browse/LOG4J2-1048 integrated.

Thank you,
Gary

> Use System.nanoTime() to measure time intervals
> -----------------------------------------------
>
>                 Key: LOG4J2-982
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-982
>             Project: Log4j 2
>          Issue Type: Improvement
>          Components: Core, Flume Appender
>            Reporter: Mikhail Mazurskiy
>            Priority: Minor
>         Attachments: LOG4J2-982-v2.patch, LOG4J2-982-v3.patch, LOG4J2-982.patch
>
>
> Unlike {{System.currentTimeMillis()}}, which can jump/slew forward and backwards, {{System.nanoTime()}} is a monotonic clock (at least it should be) and hence it should be used to measure time intervals (timeouts/delays/etc).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org