Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-13015

Use nano time in node failure detection.

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 2.8.1
    • 2.9
    • None

    Description

      Make sure node failure detection do not use:

      System.currentTimeMillis()
      and
      IgniteUtils.currentTimeMillis()
      

      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:

      ServerImpl.lastRingMsgReceivedTime

      should be nano.

      Attachments

        Issue Links

          Activity

            People

              vladsz83 Vladimir Steshin
              vladsz83 Vladimir Steshin
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 40m
                  40m