[jboss-cvs] JBossAS SVN: r82995 - branches/JBPAPP_4_2_0_GA_CP/jmx/src/main/org/jboss/mx/loading.

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

Author: jhowell at redhat.com
Date: 2009-01-16 12:53:41 -0500 (Fri, 16 Jan 2009)
New Revision: 82995

Modified:
   branches/JBPAPP_4_2_0_GA_CP/jmx/src/main/org/jboss/mx/loading/LoadMgr3.java
Log:
[JBPAPP-1491] - Added logging to identify class loader problems

Modified: branches/JBPAPP_4_2_0_GA_CP/jmx/src/main/org/jboss/mx/loading/LoadMgr3.java
===================================================================
--- branches/JBPAPP_4_2_0_GA_CP/jmx/src/main/org/jboss/mx/loading/LoadMgr3.java	2009-01-16 17:14:36 UTC (rev 82994)
+++ branches/JBPAPP_4_2_0_GA_CP/jmx/src/main/org/jboss/mx/loading/LoadMgr3.java	2009-01-16 17:53:41 UTC (rev 82995)
@@ -401,25 +401,26 @@
       }
       catch(Throwable e)
       {
-         boolean retry = e instanceof ClassCircularityError
+    	 log.error("Class loading Task Run failed with exception", e);
+    	 boolean retry = e instanceof ClassCircularityError
             || e.getClass().equals(LinkageError.class);
          int numCCE = loadTask.incNumCCE();
+         
          if( retry && numCCE <= 10 )
          {
-            /* Reschedule this task after all existing tasks to allow the
+            log.error("Rescheduling failed Class loading Task");
+        	/* Reschedule this task after all existing tasks to allow the
             current load tasks which are conflicting to complete.
             */
             try
             {
-               if( trace )
-                  log.trace("Run failed with exception", e);
                // Reschedule and update the loadTask.threadTaskCount
                scheduleTask(loadTask, ucl3, Integer.MAX_VALUE, true, trace);
             }
             catch(Throwable ex)
             {
                loadTask.setLoadError(ex);
-               log.warn("Failed to reschedule task after LinkageError", ex);               
+               log.error("Failed to reschedule task after LinkageError", ex);               
             }
             if( trace )
                log.trace("Post LinkageError state, loadTask="+loadTask);
@@ -427,8 +428,7 @@
          else
          {
             loadTask.setLoadError(e);
-            if( trace )
-               log.trace("Run failed with exception, loadTask="+loadTask, e);
+            log.error("Not resheduling failed loading task, loadTask="+loadTask, e);
          }
       }
       finally