[jboss-cvs] JBossAS SVN: r108631 - projects/jboss-jca/trunk/core/src/main/java/org/jboss/jca/core/workmanager.

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

Author: jesper.pedersen
Date: 2010-10-18 11:25:41 -0400 (Mon, 18 Oct 2010)
New Revision: 108631

Modified:
   projects/jboss-jca/trunk/core/src/main/java/org/jboss/jca/core/workmanager/WorkManagerImpl.java
Log:
Better trace logging

Modified: projects/jboss-jca/trunk/core/src/main/java/org/jboss/jca/core/workmanager/WorkManagerImpl.java
===================================================================
--- projects/jboss-jca/trunk/core/src/main/java/org/jboss/jca/core/workmanager/WorkManagerImpl.java	2010-10-18 07:55:01 UTC (rev 108630)
+++ projects/jboss-jca/trunk/core/src/main/java/org/jboss/jca/core/workmanager/WorkManagerImpl.java	2010-10-18 15:25:41 UTC (rev 108631)
@@ -237,6 +237,8 @@
                       WorkListener workListener) 
       throws WorkException
    {
+      log.tracef("doWork(%s, %s, %s, %s)", work, startTimeout, execContext, workListener);
+
       WorkException exception = null;
       WorkWrapper wrapper = null;
       try
@@ -331,6 +333,8 @@
                          WorkListener workListener) 
       throws WorkException
    {
+      log.tracef("startWork(%s, %s, %s, %s)", work, startTimeout, execContext, workListener);
+
       WorkException exception = null;
       WorkWrapper wrapper = null;
       try
@@ -431,6 +435,8 @@
                             WorkListener workListener) 
       throws WorkException
    {
+      log.tracef("scheduleWork(%s, %s, %s, %s)", work, startTimeout, execContext, workListener);
+
       WorkException exception = null;
       WorkWrapper wrapper = null;
       try
@@ -671,12 +677,12 @@
                {
                   if (trace)
                   {
-                     log.trace("Not supported work context class : " + context.getClass());
+                     log.trace("Not supported work context class : " + context.getClass().getName());
                   }
                   
                   fireWorkContextSetupFailed(context, WorkContextErrorCodes.UNSUPPORTED_CONTEXT_TYPE);
                   
-                  throw new WorkCompletedException("Unsupported WorkContext class : " + context.getClass(), 
+                  throw new WorkCompletedException("Unsupported WorkContext class : " + context.getClass().getName(), 
                       WorkContextErrorCodes.UNSUPPORTED_CONTEXT_TYPE);
                }
                // Duplicate checks
@@ -689,13 +695,13 @@
                      {
                         if (trace)
                         {
-                           log.trace("Duplicate transaction work context : " + context.getClass());
+                           log.trace("Duplicate transaction work context : " + context.getClass().getName());
                         }
 
                         fireWorkContextSetupFailed(context, WorkContextErrorCodes.DUPLICATE_CONTEXTS);
                         
                         throw new WorkCompletedException("Duplicate TransactionWorkContext class : " + 
-                            context.getClass(), WorkContextErrorCodes.DUPLICATE_CONTEXTS);
+                           context.getClass().getName(), WorkContextErrorCodes.DUPLICATE_CONTEXTS);
                      }
                      else
                      {
@@ -709,13 +715,14 @@
                      {
                         if (trace)
                         {
-                           log.trace("Duplicate security work context : " + context.getClass());
+                           log.trace("Duplicate security work context : " + context.getClass().getName());
                         }
                         
                         fireWorkContextSetupFailed(context, WorkContextErrorCodes.DUPLICATE_CONTEXTS);
 
-                        throw new WorkCompletedException("Duplicate SecurityWorkContext class : " + context.getClass(), 
-                              WorkContextErrorCodes.DUPLICATE_CONTEXTS);
+                        throw new WorkCompletedException("Duplicate SecurityWorkContext class : " + 
+                                                         context.getClass().getName(), 
+                                                         WorkContextErrorCodes.DUPLICATE_CONTEXTS);
                      }
                      else
                      {
@@ -729,13 +736,14 @@
                      {
                         if (trace)
                         {
-                           log.trace("Duplicate hint work context : " + context.getClass());
+                           log.trace("Duplicate hint work context : " + context.getClass().getName());
                         }
 
                         fireWorkContextSetupFailed(context, WorkContextErrorCodes.DUPLICATE_CONTEXTS);
                         
-                        throw new WorkCompletedException("Duplicate HintWorkContext class : " + context.getClass(), 
-                              WorkContextErrorCodes.DUPLICATE_CONTEXTS);
+                        throw new WorkCompletedException("Duplicate HintWorkContext class : " + 
+                                                         context.getClass().getName(), 
+                                                         WorkContextErrorCodes.DUPLICATE_CONTEXTS);
                      }
                      else
                      {
@@ -747,8 +755,9 @@
                   {
                      fireWorkContextSetupFailed(context, WorkContextErrorCodes.UNSUPPORTED_CONTEXT_TYPE);
                      
-                     throw new WorkCompletedException("Unsupported WorkContext class : " + context.getClass(), 
-                           WorkContextErrorCodes.UNSUPPORTED_CONTEXT_TYPE);
+                     throw new WorkCompletedException("Unsupported WorkContext class : " + 
+                                                      context.getClass().getName(), 
+                                                      WorkContextErrorCodes.UNSUPPORTED_CONTEXT_TYPE);
                   }
                }