Uploaded image for project: 'Bookkeeper'
  1. Bookkeeper
  2. BOOKKEEPER-327

System.currentTimeMillis usage in BookKeeper

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 4.0.0, 4.1.0
    • 4.2.0, 4.1.1
    • None
    • None

    Description

      The following exception occured in the bookie statistics logic due to the System time changes. In our bookie cluster its running a periodic syncup scripts just to unify the SystemTime in all the machines. This is causing the problem and resulting ArrayIndexOutOfBoundException.

      Exception in thread "BookieJournal-3181" java.lang.ArrayIndexOutOfBoundsException: -423
      at org.apache.bookkeeper.proto.BKStats$OpStats.updateLatency(BKStats.java:126)
      at org.apache.bookkeeper.proto.BookieServer.writeComplete(BookieServer.java:655)
      at org.apache.bookkeeper.bookie.Journal.run(Journal.java:507)
      

      This jira is raised to discuss whether to use System.nanoTime() instead of System.currentTimeMillis()

      Attachments

        1. BOOKKEEPER-327.patch
          21 kB
          Uma Maheswara Rao G
        2. BOOKKEEPER-327.patch
          18 kB
          Uma Maheswara Rao G
        3. BOOKKEEPER-327-1.patch
          21 kB
          Uma Maheswara Rao G
        4. BOOKKEEPER-327-2.patch
          21 kB
          Uma Maheswara Rao G
        5. BOOKKEEPER-327-Branch4.1.patch
          20 kB
          Flavio Paiva Junqueira

        Issue Links

          Activity

            People

              umamaheswararao Uma Maheswara Rao G
              rakeshr Rakesh Radhakrishnan
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: