[jboss-cvs] JBossAS SVN: r111978 - in projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc: jdk6 and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: jesper.pedersen
Date: 2011-08-10 11:16:57 -0400 (Wed, 10 Aug 2011)
New Revision: 111978

Modified:
   projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/CachedPreparedStatement.java
   projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/jdk6/CachedPreparedStatementJDK6.java
Log:
Trace preparedstatement close methods

Modified: projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/CachedPreparedStatement.java
===================================================================
--- projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/CachedPreparedStatement.java	2011-08-10 12:58:20 UTC (rev 111977)
+++ projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/CachedPreparedStatement.java	2011-08-10 15:16:57 UTC (rev 111978)
@@ -38,6 +38,8 @@
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import org.jboss.logging.Logger;
+
 /**
  * Wrapper class for cached PreparedStatements.  Keeps a refcount.  When this refcount reaches 0,
  * it will close ps.
@@ -49,6 +51,8 @@
 public abstract class CachedPreparedStatement extends JBossWrapper implements PreparedStatement
 {
    private static final long serialVersionUID = 2085461257386274373L;
+   protected static Logger log = Logger.getLogger(CachedPreparedStatement.class);
+   protected static boolean trace = log.isTraceEnabled();
    
    private PreparedStatement ps;
    private AtomicBoolean cached = new AtomicBoolean(true);
@@ -437,6 +441,22 @@
    }
 
    /**
+    * Close prepared statement
+    * @exception SQLException Thrown if an error occurs
+    */
+   private void closePreparedStatement() throws SQLException
+   {
+      if (trace)
+      {
+         Throwable t = new Throwable("PreparedStatement.close() called");
+         t.setStackTrace(Thread.currentThread().getStackTrace());
+         log.trace(t.getMessage(), t);
+      }
+
+      ps.close();
+   }
+
+   /**
     * Age out
     * @exception SQLException Thrown if an error occurs
     */
@@ -444,7 +464,7 @@
    {
       cached.set(false);
       if (inUse.get() == 0)
-         ps.close();
+         closePreparedStatement();
    }
 
    /**
@@ -457,7 +477,7 @@
       {
          if (!cached.get())
          {
-            ps.close();
+            closePreparedStatement();
          }
          else
          {

Modified: projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/jdk6/CachedPreparedStatementJDK6.java
===================================================================
--- projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/jdk6/CachedPreparedStatementJDK6.java	2011-08-10 12:58:20 UTC (rev 111977)
+++ projects/jboss-jca/trunk/adapters/src/main/java/org/jboss/jca/adapters/jdbc/jdk6/CachedPreparedStatementJDK6.java	2011-08-10 15:16:57 UTC (rev 111978)
@@ -33,7 +33,7 @@
 import java.sql.SQLXML;
 
 /**
- * CachedPreparedStatementJDK5.
+ * CachedPreparedStatementJDK6.
  * 
  * @author <a href="adrian at jboss.com">Adrian Brock</a>
  * @version $Revision: 85945 $