Uploaded image for project: 'Blazegraph (by SYSTAP)'
  1. Blazegraph (by SYSTAP)
  2. BLZG-714

Row store read/update not isolated on Journal

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Affects Version/s: BIGDATA_RELEASE_1_2_2
    • Fix Version/s: None
    • Component/s: Journal

      Description

      The row store operations are not isolated on the journal. These operations were written to depend on the ConcurrencyManager and are ACID on a cluster, but they are not run on the ConcurrencyManager for a Journal and hence lack appropriate isolation. This problem exists in all versions, but rarely shows up because the row store is mainly used for KB creation metadata and we rarely have concurrent create / discover / destroy operations for KBs.

        Activity

        beebs Brad Bebee created issue -
        Hide
        bryanthompson bryanthompson added a comment -

        GlobalRowStoreHelper has been modified to use AbstractRelation.getIndex(...) to wrap a live index view with an UnisolatedReadWriteIndex. This is the same code path that is used to protect normal named indices used by relations on the Journal. This code change fixes problems that had been observed with TestConcurrentKBCreate that were clearly related to the lack of isolation for the live view of the global row store. However, there is still a test failure mode for TestConcurrentKBCreate that appears to be related to the default resource locator cache.

        Committed revision r6706.

        Show
        bryanthompson bryanthompson added a comment - GlobalRowStoreHelper has been modified to use AbstractRelation.getIndex(...) to wrap a live index view with an UnisolatedReadWriteIndex. This is the same code path that is used to protect normal named indices used by relations on the Journal. This code change fixes problems that had been observed with TestConcurrentKBCreate that were clearly related to the lack of isolation for the live view of the global row store. However, there is still a test failure mode for TestConcurrentKBCreate that appears to be related to the default resource locator cache. Committed revision r6706.
        beebs Brad Bebee made changes -
        Field Original Value New Value
        Workflow Trac Import v2 [ 12550 ] Trac Import v3 [ 14101 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v3 [ 14101 ] Trac Import v4 [ 15430 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v4 [ 15430 ] Trac Import v5 [ 16816 ]
        beebs Brad Bebee made changes -
        Labels Issue_patch_20150625
        beebs Brad Bebee made changes -
        Status Closed - Won't Fix [ 6 ] Open [ 1 ]
        beebs Brad Bebee made changes -
        Status Open [ 1 ] Accepted [ 10101 ]
        beebs Brad Bebee made changes -
        Status Accepted [ 10101 ] In Progress [ 3 ]
        beebs Brad Bebee made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        beebs Brad Bebee made changes -
        Status Resolved [ 5 ] In Review [ 10100 ]
        beebs Brad Bebee made changes -
        Resolution Fixed [ 1 ] Done [ 10000 ]
        Status In Review [ 10100 ] Done [ 10000 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v5 [ 16816 ] Trac Import v6 [ 18033 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v6 [ 18033 ] Trac Import v7 [ 19430 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v7 [ 19430 ] Trac Import v8 [ 21051 ]

          People

          • Assignee:
            bryanthompson bryanthompson
            Reporter:
            bryanthompson bryanthompson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: