Uploaded image for project: 'Karaf'
  1. Karaf
  2. KARAF-689

Karaf instance can wrongly obtain a lock when the locking table is empty on a Oracle DB


    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.3
    • Fix Version/s: 2.1.6, 2.2.2, 3.0.0
    • Component/s: karaf-core
    • Labels:
    • Environment:

      Oracle Database 11g Enterprise Edition Release - 64bit Production


      When running Karaf in Master/Slave mode a Karaf instance can wrongly obtain a lock when the locking table is empty on a Oracle DB.
      Karaf creates a lock table and a lock record in de OracleDB upon first usage. When the lock record is removed afterwards without dropping the table then the locking mechanisme does not work anymore and any instance can wrongly obtain a lock.

      This is not an issue for other database systems, because the implementation for other DB systems periodically tries to update the lock record and verify how many records are updated. According to the documentation in the code the "update" functionality has been removed for Oracle because it can exhaust the Oracle UNDO log.

      I created a patch which does not use the "update" functionality, but executes an extra check after we acquired the lock. It checks if the SELECT FOR UPDATE used for the lock has not been executed on an empty selection, because when the selection is empty then the locking functionality will not work as expected.

      Can you apply this patch to the branches karaf-2.1.x and karaf-2.2.x and the trunk. I looked at the SVN history of the files I have modified and noticed that they are the same in both branches and the trunk.
      The patch has been created against the karaf-2.1.x branch.


        1. patch.txt
          9 kB
          Ton Swieb
        2. patch.txt
          9 kB
          Ton Swieb



            • Assignee:
              jgoodyear Jamie goodyear
              tonswieb Ton Swieb
            • Votes:
              0 Vote for this issue
              0 Start watching this issue


              • Created: