OpenJPA
  1. OpenJPA
  2. OPENJPA-2134

Do not close Connections when ConnectionRetainMode is set to "always"

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.1, 2.3.0
    • Fix Version/s: 2.2.1, 2.3.0
    • Component/s: jdbc, performance
    • Labels:
      None

      Description

      Currently, during the prepareForPooling processing, we are closing the Connection before allowing the EM to be put placed into a Container's EM Pool. That seems to be overkill, especially if the ConnectionRetainMode is set to "always". We'll use this Issue to at least improve this processing when ConnectionRetainMode is set to "always". But, this whole prepareForPooling might be unnecessary overhead. Only correcting one issue at a time...

        Activity

        Kevin Sutter created issue -
        Kevin Sutter made changes -
        Field Original Value New Value
        Status Open [ 1 ] In Progress [ 3 ]
        Kevin Sutter made changes -
        Fix Version/s 2.3.0 [ 12319463 ]
        Fix Version/s 2.2.1 [ 12319943 ]
        Albert Lee made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Albert Lee made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        ASF subversion and git services added a comment -

        Commit 1564976 from Rick Curtis in branch 'openjpa/trunk'
        [ https://svn.apache.org/r1564976 ]

        OPENJPA-2134: Deprecate EntityManagerImpl.prepareForPooling().

        Show
        ASF subversion and git services added a comment - Commit 1564976 from Rick Curtis in branch 'openjpa/trunk' [ https://svn.apache.org/r1564976 ] OPENJPA-2134 : Deprecate EntityManagerImpl.prepareForPooling().
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        8s 1 Kevin Sutter 21/Feb/12 15:58
        In Progress In Progress Resolved Resolved
        231d 2h 20m 1 Albert Lee 09/Oct/12 19:19
        Resolved Resolved Closed Closed
        4s 1 Albert Lee 09/Oct/12 19:19

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development