[jboss-cvs] JBossAS SVN: r108097 - projects/cluster/ha-server-cache-ispn/trunk/src/main/java/org/jboss/ha/jndi/ispn.

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

Author: smarlow at redhat.com
Date: 2010-09-09 18:32:51 -0400 (Thu, 09 Sep 2010)
New Revision: 108097

Modified:
   projects/cluster/ha-server-cache-ispn/trunk/src/main/java/org/jboss/ha/jndi/ispn/DistributedTreeManager.java
Log:
Eliminate reference to Infinispan CacheException when throwing NamingException.  More trace logging

Modified: projects/cluster/ha-server-cache-ispn/trunk/src/main/java/org/jboss/ha/jndi/ispn/DistributedTreeManager.java
===================================================================
--- projects/cluster/ha-server-cache-ispn/trunk/src/main/java/org/jboss/ha/jndi/ispn/DistributedTreeManager.java	2010-09-09 21:10:09 UTC (rev 108096)
+++ projects/cluster/ha-server-cache-ispn/trunk/src/main/java/org/jboss/ha/jndi/ispn/DistributedTreeManager.java	2010-09-09 22:32:51 UTC (rev 108097)
@@ -188,9 +188,14 @@
       }
       catch (CacheException ce)
       {
+         if (LOG.isTraceEnabled())
+         {
+            LOG.trace(ce.getMessage(), ce);
+         }
+
          // don't chain CacheException since Infinispan may not be on remote client's classpath
          NamingException ne = new NamingException(ce.getClass().getName() + ": " + ce.getMessage());
-         ne.setStackTrace(ce.getStackTrace());
+         ne.fillInStackTrace();
          throw ne;
       }
       
@@ -261,9 +266,13 @@
       }
       catch (CacheException ce)
       {
+         if (LOG.isTraceEnabled())
+         {
+            LOG.trace(ce.getMessage(), ce);
+         }
          // don't chain CacheException since Infinispan may not be on remote client's classpath
          NamingException ne = new NamingException(ce.getClass().getName() + ": " + ce.getMessage());
-         ne.setStackTrace(ce.getStackTrace());
+         ne.fillInStackTrace();
          throw ne;
       }
    
@@ -292,9 +301,13 @@
       }
       catch (CacheException ce)
       {
+         if (LOG.isTraceEnabled())
+         {
+            LOG.trace(ce.getMessage(), ce);
+         }
          // don't chain CacheException since JBoss Cache may not be on remote client's classpath
          NamingException ne = new NamingException(ce.getClass().getName() + ": " + ce.getMessage());
-         ne.setStackTrace(ce.getStackTrace());
+         ne.fillInStackTrace();
          throw ne;
       }
    }
@@ -365,9 +378,14 @@
       }
       catch (CacheException ce)
       {
+         if (LOG.isTraceEnabled())
+         {
+            LOG.trace(ce.getMessage(), ce);
+         }
+
          // don't chain CacheException since JBoss Cache may not be on remote client's classpath
          NamingException ne = new NamingException(ce.getClass().getName() + ": " + ce.getMessage());
-         ne.setStackTrace(ce.getStackTrace());
+         ne.fillInStackTrace();
          throw ne;
       }
    }
@@ -441,9 +459,14 @@
       }
       catch (CacheException ce)
       {
+         if (LOG.isTraceEnabled())
+         {
+            LOG.trace(ce.getMessage(), ce);
+         }
+
          // don't chain CacheException since Infinispan may not be on remote client's classpath
          NamingException ne = new NamingException(ce.getClass().getName() + ": " + ce.getMessage());
-         ne.setStackTrace(ce.getStackTrace());
+         ne.fillInStackTrace();
          throw ne;
       }
    }
@@ -498,9 +521,13 @@
       }
       catch (CacheException ce)
       {
+         if (LOG.isTraceEnabled())
+         {
+            LOG.trace(ce.getMessage(), ce);
+         }
          // don't chain CacheException since JBoss Cache may not be on remote client's classpath
          NamingException ne = new NamingException(ce.getClass().getName() + ": " + ce.getMessage());
-         ne.setStackTrace(ce.getStackTrace());
+         ne.fillInStackTrace();
          throw ne;
       }