Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-6137

Remove call to getHeadNodeState in LockBasedScheduler constructor

    XMLWordPrintableJSON

Details

    • Task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.7.3, 1.8.0
    • segment-tar
    • None

    Description

      When creating the ChangeDispatcher instance in the constructor of LockBasedScheduler we should avoid the call to LockBasedScheduler.getHeadNodeState(). This is a call to an overridable method at construction time, which can be troublesome as the instance in not fully initialised yet. On top of that the default implementation might have side effects through its call to refreshHead. We should probably just use head.get() to initialise that ChangeDispatcher instance.

      Attachments

        Activity

          People

            adulceanu Andrei Dulceanu
            adulceanu Andrei Dulceanu
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: