You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pp...@apache.org on 2010/03/30 18:33:17 UTC

svn commit: r929184 - /openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java

Author: ppoddar
Date: Tue Mar 30 16:33:16 2010
New Revision: 929184

URL: http://svn.apache.org/viewvc?rev=929184&view=rev
Log:
OPENJPA-1294: New connection is not reference counted

Modified:
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java?rev=929184&r1=929183&r2=929184&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java Tue Mar 30 16:33:16 2010
@@ -975,7 +975,7 @@ public class JDBCStoreManager 
     
     public Connection getNewConnection() {
         try {
-            return connectInternal();
+            return _ds.getConnection();
         } catch (SQLException e) {
             throw SQLExceptions.getStore(e, _dict);
         }

Re: svn commit: r929184 - /openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java

Posted by Donald Woods <dw...@apache.org>.
Pinaki, should this also be included in the 2.0.0 release?  The issue
was marked as being fixed in 2.0.0-beta.


-Donald


On 3/30/10 12:33 PM, ppoddar@apache.org wrote:
> Author: ppoddar
> Date: Tue Mar 30 16:33:16 2010
> New Revision: 929184
> 
> URL: http://svn.apache.org/viewvc?rev=929184&view=rev
> Log:
> OPENJPA-1294: New connection is not reference counted
> 
> Modified:
>     openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
> 
> Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
> URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java?rev=929184&r1=929183&r2=929184&view=diff
> ==============================================================================
> --- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java (original)
> +++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java Tue Mar 30 16:33:16 2010
> @@ -975,7 +975,7 @@ public class JDBCStoreManager 
>      
>      public Connection getNewConnection() {
>          try {
> -            return connectInternal();
> +            return _ds.getConnection();
>          } catch (SQLException e) {
>              throw SQLExceptions.getStore(e, _dict);
>          }
> 
> 
>