Jackrabbit Content Repository
  1. Jackrabbit Content Repository
  2. JCR-2613

NoSuchItemStateException on checkin after removeVersion in XA Environment

    Details

      Description

      After removing a version, a checkin on the same node in a different transaction (with a different session) fails.
      The NoSuchItemStateException refer to the uuid of the previously removed version.
      I'll attach a test demonstrating the problem.

      1. RemoveAndCheckinXATest.java
        1 kB
        Tom Quellenberg
      2. JCR-2613.patch
        0.7 kB
        Tom Quellenberg

        Activity

        Hide
        Tom Quellenberg added a comment -

        I think a node.store() is missing in InternalVersionHistoryImpl.removeVersion when no references to the version history were found. Without the store operation the cache in the LocalItemStateManager is not updated.
        See proposed patch (based on the 1.6-branch)
        I suppose the bug was introduced by JCR-134 and affects all version since 1.6.0

        Show
        Tom Quellenberg added a comment - I think a node.store() is missing in InternalVersionHistoryImpl.removeVersion when no references to the version history were found. Without the store operation the cache in the LocalItemStateManager is not updated. See proposed patch (based on the 1.6-branch) I suppose the bug was introduced by JCR-134 and affects all version since 1.6.0
        Hide
        Tobias Bocanegra added a comment -

        fixed in 1.6 branch by revision 936666

        Show
        Tobias Bocanegra added a comment - fixed in 1.6 branch by revision 936666
        Hide
        Tobias Bocanegra added a comment -

        also fixed in trunk by revision 936668.

        Show
        Tobias Bocanegra added a comment - also fixed in trunk by revision 936668.
        Hide
        Jukka Zitting added a comment -

        Merged to the 2.1 branch in revision 982266 and to the 2.0 branch in revision 982270.

        Show
        Jukka Zitting added a comment - Merged to the 2.1 branch in revision 982266 and to the 2.0 branch in revision 982270.

          People

          • Assignee:
            Tobias Bocanegra
            Reporter:
            Tom Quellenberg
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development