Uploaded image for project: '[Read Only] - Hippo Repository'
  1. [Read Only] - Hippo Repository
  2. REPO-1645

Patched Jackrabbit LockManagerImpl is not fully honoring the JCR API

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Normal
    • Resolution: Fixed
    • None
    • 5.0.0
    • None
    • None
    • 1
    • Platform Sprint 153

    Description

      REPO-1381 intruced a new 'lazy' or 'just-in-time' expiration handling of JCR locks, thereby no longer needing the Jackrabbit default background thread monitoring and removing possibly expired locks.

      However, that customization currently does not honor a few JCR Lock API method usages: #isLive() and #getSecondsRemaining(), and which actually fail when running the Jackrabbit integration tests.

      Furthermore the internal (within LockManagerImpl) boolean #unlockIfExpired(LockInfo) method is incorrectly used in several places to (also) check if a lock is 'live'. Which may cause a false positive effect.

      With this improvement issue those problems need to be solved and replace the current used REPO-1381 'patch' in building the Hippo specific Jackrabbit version, starting with Jackrabbit-2.14.0. 

      Attachments

        Issue Links

          is a part of

          Improvement - An improvement or enhancement to an existing feature or task. REPO-1644 Upgrade to Jackrabbit 2.14.0

          • Normal - Normal priority, tasks to be done before next intermediate release
          • Closed
          is a result of

          Bug - A problem which impairs or prevents the functions of the product. REPO-1381 Concurrent session access in Jackrabbit's LockManager timeout handler

          • Normal - Normal priority, tasks to be done before next intermediate release
          • Closed
          is cloned by

          Improvement - An improvement or enhancement to an existing feature or task. REPO-1674 [Backport to CMS 11.1, 11.2 and 10.2] Patched Jackrabbit LockManagerImpl is not fully honoring the JCR API

          • Normal - Normal priority, tasks to be done before next intermediate release
          • Closed

          Activity

            People

              Unassigned Unassigned
              adouma Ate Douma (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: