[ https://issues.apache.org/jira/browse/IGNITE-13015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17147624#comment-17147624 ]
Anton Vinogradov commented on IGNITE-13015: ------------------------------------------- Merged to master. Thanks for your contribution. > Use nano time in node failure detection. > ---------------------------------------- > > Key: IGNITE-13015 > URL: https://issues.apache.org/jira/browse/IGNITE-13015 > Project: Ignite > Issue Type: Improvement > Affects Versions: 2.8.1 > Reporter: Vladimir Steshin > Assignee: Vladimir Steshin > Priority: Minor > Labels: iep-45 > Fix For: 2.9 > > Time Spent: 40m > Remaining Estimate: 0h > > Make sure node failure detection do not use: > {code:java} > System.currentTimeMillis() > and > IgniteUtils.currentTimeMillis() > {code} > We should use nano time instead. Disadventages of current impl.: > 1) System time has no quarantine of strict forward movement. System time > can be adjusted, synchronized by NTP as example. This can lead to incorrect > and negative delays. > 2) IgniteUtils.currentTimeMillis() is granulated by 10ms > *To fix*: > {code:java}ServerImpl.lastRingMsgReceivedTime{code} should be nano. -- This message was sent by Atlassian Jira (v8.3.4#803005)