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

Gary Gregory edited comment on LOG4J2-982 at 3/26/15 5:10 PM:
--------------------------------------------------------------

OK, it is sounds like the patch makes this deadlock surface for me and not you. 
Let's see what Ralph and/or Remko see when they take a look.


was (Author: garydgregory):
OK, it is sounds like the patch makes this deadlock surface from me and not 
you. Let's see what Ralph and/or Remko see when they take a look.

> 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.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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to