[jboss-cvs] JBossAS SVN: r79016 - branches/JBPAPP_4_2_0_GA_CP/server/src/main/org/jboss/tm/usertx/client.

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

Author: galder.zamarreno at jboss.com
Date: 2008-10-02 06:04:09 -0400 (Thu, 02 Oct 2008)
New Revision: 79016

Modified:
   branches/JBPAPP_4_2_0_GA_CP/server/src/main/org/jboss/tm/usertx/client/ClientUserTransaction.java
Log:
[JBPAPP-1245] Add trace logging for cause exceptions when these are not propagated back. This helps debugging. 

Modified: branches/JBPAPP_4_2_0_GA_CP/server/src/main/org/jboss/tm/usertx/client/ClientUserTransaction.java
===================================================================
--- branches/JBPAPP_4_2_0_GA_CP/server/src/main/org/jboss/tm/usertx/client/ClientUserTransaction.java	2008-10-02 09:59:59 UTC (rev 79015)
+++ branches/JBPAPP_4_2_0_GA_CP/server/src/main/org/jboss/tm/usertx/client/ClientUserTransaction.java	2008-10-02 10:04:09 UTC (rev 79016)
@@ -46,7 +46,6 @@
 
 import org.jboss.tm.usertx.interfaces.UserTransactionSession;
 import org.jboss.tm.usertx.interfaces.UserTransactionSessionFactory;
-import org.jboss.util.Strings;
 import org.jboss.logging.Logger;
 import org.jboss.naming.NamingContextFactory;
 
@@ -128,10 +127,12 @@
       {
          // destroy session gone bad.
          destroySession();
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
       catch (Exception e)
       {
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
    }
@@ -180,10 +181,12 @@
       {
          // destroy session gone bad.
          destroySession();
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
       catch (Exception e)
       {
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
    }
@@ -216,10 +219,12 @@
       {
          // destroy session gone bad.
          destroySession();
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
       catch (Exception e)
       {
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
    }
@@ -246,10 +251,12 @@
       {
          // destroy session gone bad.
          destroySession();
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
       catch (Exception e)
       {
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
    }
@@ -277,10 +284,12 @@
       {
          // destroy session gone bad.
          destroySession();
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
       catch (Exception e)
       {
+         logCauseException(e);
          throw new SystemException(e.toString());
       }
    }
@@ -414,6 +423,13 @@
       return ret;
    }
 
+   private void logCauseException(Exception e)
+   {
+      if (trace)
+      {
+         log.trace("Logging cause exception", e);         
+      }
+   }
 
    // Inner classes -------------------------------------------------