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

Details

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

    Description

      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.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD