Uploaded image for project: 'Magnolia Soft Locking Module'
  1. Magnolia Soft Locking Module
  2. MGNLSLOCK-4

Modifications to map of concurrently editing users should be synchronized

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 1.0.2
    • 1.0
    • None

      Even though the map is a ConcurrentMap, DefaultSoftLockingSupport first probes the value, then checks if its null to create a new map to put in. This is a race condition that can result in lost updates.

      A lock for modification should be introduced and used in addCurrentUserToEditingList(). The lock should also be held in removeCurrentUserFromEditingList() allowing for it to probe if the list becomes empty and if it is remove it from the map.

        Acceptance criteria

              fgrilli Federico Grilli
              tmattsson Tobias Mattsson
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD