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

Remove call to getHeadNodeState in LockBasedScheduler constructor

    XMLWordPrintableJSON

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7.3, 1.8.0
    • Component/s: segment-tar
    • Labels:
      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

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

              Dates

              • Created:
                Updated:
                Resolved: