OpenJPA
  1. OpenJPA
  2. OPENJPA-2437

transactional listeners added too late to observe begin event

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.1.1, 2.2.3, 2.3.0
    • Component/s: kernel
    • Labels:
      None
    1. tx-listener.patch
      10 kB
      Romain Manni-Bucau
    2. openjpa-listeners.patch
      2 kB
      Romain Manni-Bucau

      Issue Links

        Activity

        Romain Manni-Bucau created issue -
        Romain Manni-Bucau made changes -
        Field Original Value New Value
        Link This issue blocks TOMEE-1056 [ TOMEE-1056 ]
        Romain Manni-Bucau made changes -
        Attachment openjpa-listeners.patch [ 12608021 ]
        Hide
        Rick Curtis added a comment -

        Romain Manni-Bucau – The proposed change looks good by me, but is it possible to come up with a small OpenJPA unit test that would expose this bug to ensure that we don't regress this behavior in the future?

        Show
        Rick Curtis added a comment - Romain Manni-Bucau – The proposed change looks good by me, but is it possible to come up with a small OpenJPA unit test that would expose this bug to ensure that we don't regress this behavior in the future?
        Hide
        Romain Manni-Bucau added a comment -

        I'll do ASAP (surely next week)

        Show
        Romain Manni-Bucau added a comment - I'll do ASAP (surely next week)
        Hide
        Romain Manni-Bucau added a comment -

        adding a test to the patch

        Show
        Romain Manni-Bucau added a comment - adding a test to the patch
        Romain Manni-Bucau made changes -
        Attachment tx-listener.patch [ 12608160 ]
        Mark Struberg made changes -
        Assignee Mark Struberg [ struberg ]
        Mark Struberg made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Mark Struberg added a comment -

        fix applied + cleaned up + test case rewritten to openjpa style. Now running all tests before committing.

        Show
        Mark Struberg added a comment - fix applied + cleaned up + test case rewritten to openjpa style. Now running all tests before committing.
        Hide
        ASF subversion and git services added a comment -

        Commit 1536595 from Mark Struberg in branch 'openjpa/trunk'
        [ https://svn.apache.org/r1536595 ]

        OPENJPA-2437 transactional listeners added too late to observe begin event

        txs 2 rmannibucau for the fix.
        Patch applied with cleanup and tests pimped.

        Show
        ASF subversion and git services added a comment - Commit 1536595 from Mark Struberg in branch 'openjpa/trunk' [ https://svn.apache.org/r1536595 ] OPENJPA-2437 transactional listeners added too late to observe begin event txs 2 rmannibucau for the fix. Patch applied with cleanup and tests pimped.
        Hide
        ASF subversion and git services added a comment -

        Commit 1536596 from Mark Struberg in branch 'openjpa/branches/2.3.x'
        [ https://svn.apache.org/r1536596 ]

        OPENJPA-2437 transactional listeners added too late to observe begin event

        txs 2 rmannibucau for the fix.
        Patch applied with cleanup and tests pimped.

        Show
        ASF subversion and git services added a comment - Commit 1536596 from Mark Struberg in branch 'openjpa/branches/2.3.x' [ https://svn.apache.org/r1536596 ] OPENJPA-2437 transactional listeners added too late to observe begin event txs 2 rmannibucau for the fix. Patch applied with cleanup and tests pimped.
        Mark Struberg made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 2.3.0 [ 12319463 ]
        Resolution Fixed [ 1 ]
        Hide
        ASF subversion and git services added a comment -

        Commit 1584153 from Heath Thomann in branch 'openjpa/branches/2.2.1.x'
        [ https://svn.apache.org/r1584153 ]

        OPENJPA-2437: transactional listeners added too late to observe begin event - back ported Mark Struber's changes to 2.2.1.x.

        Show
        ASF subversion and git services added a comment - Commit 1584153 from Heath Thomann in branch 'openjpa/branches/2.2.1.x' [ https://svn.apache.org/r1584153 ] OPENJPA-2437 : transactional listeners added too late to observe begin event - back ported Mark Struber's changes to 2.2.1.x.
        Hide
        ASF subversion and git services added a comment -

        Commit 1584156 from Heath Thomann in branch 'openjpa/branches/2.2.x'
        [ https://svn.apache.org/r1584156 ]

        OPENJPA-2437: transactional listeners added too late to observe begin event - back ported Mark Struber's changes to 2.2.x.

        Show
        ASF subversion and git services added a comment - Commit 1584156 from Heath Thomann in branch 'openjpa/branches/2.2.x' [ https://svn.apache.org/r1584156 ] OPENJPA-2437 : transactional listeners added too late to observe begin event - back ported Mark Struber's changes to 2.2.x.
        Heath Thomann made changes -
        Fix Version/s 2.2.1.1 [ 12323484 ]
        Fix Version/s 2.2.3 [ 12324305 ]
        Mark Struberg made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        17d 4h 53m 1 Mark Struberg 28/Oct/13 21:03
        In Progress In Progress Resolved Resolved
        10h 55m 1 Mark Struberg 29/Oct/13 07:58
        Resolved Resolved Closed Closed
        585d 13h 21m 1 Mark Struberg 06/Jun/15 22:19

          People

          • Assignee:
            Mark Struberg
            Reporter:
            Romain Manni-Bucau
          • Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development