[jboss-svn-commits] JBoss Common SVN: r2789 - in common-core/trunk/src: test/java/org/jboss/test/util/test/concurrent and 1 other directory.

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

Author: adrian at jboss.org
Date: 2008-03-31 14:20:26 -0400 (Mon, 31 Mar 2008)
New Revision: 2789

Modified:
   common-core/trunk/src/main/java/org/jboss/util/threadpool/BasicThreadPool.java
   common-core/trunk/src/main/java/org/jboss/util/threadpool/ThreadPoolFullException.java
   common-core/trunk/src/test/java/org/jboss/test/util/test/concurrent/ThreadPoolInterruptedThreadUnitTestCase.java
Log:
[JBCOMMON-47] - Show the full exception

Modified: common-core/trunk/src/main/java/org/jboss/util/threadpool/BasicThreadPool.java
===================================================================
--- common-core/trunk/src/main/java/org/jboss/util/threadpool/BasicThreadPool.java	2008-03-31 18:17:26 UTC (rev 2788)
+++ common-core/trunk/src/main/java/org/jboss/util/threadpool/BasicThreadPool.java	2008-03-31 18:20:26 UTC (rev 2789)
@@ -489,7 +489,7 @@
       }
       catch (Throwable t)
       {
-         wrapper.rejectTask(new ThreadPoolFullException(t.toString()));
+         wrapper.rejectTask(new ThreadPoolFullException("Error scheduling work: " + wrapper, t));
       }
    }
 

Modified: common-core/trunk/src/main/java/org/jboss/util/threadpool/ThreadPoolFullException.java
===================================================================
--- common-core/trunk/src/main/java/org/jboss/util/threadpool/ThreadPoolFullException.java	2008-03-31 18:17:26 UTC (rev 2788)
+++ common-core/trunk/src/main/java/org/jboss/util/threadpool/ThreadPoolFullException.java	2008-03-31 18:20:26 UTC (rev 2789)
@@ -31,14 +31,7 @@
 {
    /** The serialVersionUID */
    private static final long serialVersionUID = 1L;
-   // Constants -----------------------------------------------------
 
-   // Attributes ----------------------------------------------------
-
-   // Static --------------------------------------------------------
-
-   // Constructors --------------------------------------------------
-
    /**
     * Create a new ThreadPoolFullException
     */
@@ -56,15 +49,15 @@
    {
       super(message);
    }
-
-   // Public --------------------------------------------------------
-
-
-   // Package protected ---------------------------------------------
-
-   // Protected -----------------------------------------------------
-
-   // Private -------------------------------------------------------
-
-   // Inner classes -------------------------------------------------
+   
+   /**
+    * Create a new ThreadPoolFullException.
+    * 
+    * @param message the message
+    * @param t the throwable
+    */
+   public ThreadPoolFullException(String message, Throwable t)
+   {
+      super(message, t);
+   }
 }

Modified: common-core/trunk/src/test/java/org/jboss/test/util/test/concurrent/ThreadPoolInterruptedThreadUnitTestCase.java
===================================================================
--- common-core/trunk/src/test/java/org/jboss/test/util/test/concurrent/ThreadPoolInterruptedThreadUnitTestCase.java	2008-03-31 18:17:26 UTC (rev 2788)
+++ common-core/trunk/src/test/java/org/jboss/test/util/test/concurrent/ThreadPoolInterruptedThreadUnitTestCase.java	2008-03-31 18:20:26 UTC (rev 2789)
@@ -21,20 +21,13 @@
  */
 package org.jboss.test.util.test.concurrent;
 
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.HashSet;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
-import java.util.concurrent.atomic.AtomicBoolean;
 
-import org.jboss.util.threadpool.BasicTaskWrapper;
-import org.jboss.util.threadpool.BasicThreadPool;
-import org.jboss.util.threadpool.RunnableTaskWrapper;
-import org.jboss.util.threadpool.Task;
-import org.jboss.logging.Logger;
 import junit.framework.TestCase;
 
+import org.jboss.util.threadpool.BasicThreadPool;
+
 /**
  * Tests of thread pool with Tasks added to the pool
  *