[jboss-svn-commits] JBL Code SVN: r26608 - labs/jbossesb/branches/JBESB_4_4_GA_CP/product/rosetta/src/org/jboss/soa/esb/client.

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

Author: kevin.conner at jboss.com
Date: 2009-05-19 05:24:43 -0400 (Tue, 19 May 2009)
New Revision: 26608

Modified:
   labs/jbossesb/branches/JBESB_4_4_GA_CP/product/rosetta/src/org/jboss/soa/esb/client/ServiceInvoker.java
Log:
Add isDebugEnabled checks and include exception: JBESB-2493

Modified: labs/jbossesb/branches/JBESB_4_4_GA_CP/product/rosetta/src/org/jboss/soa/esb/client/ServiceInvoker.java
===================================================================
--- labs/jbossesb/branches/JBESB_4_4_GA_CP/product/rosetta/src/org/jboss/soa/esb/client/ServiceInvoker.java	2009-05-19 08:48:55 UTC (rev 26607)
+++ labs/jbossesb/branches/JBESB_4_4_GA_CP/product/rosetta/src/org/jboss/soa/esb/client/ServiceInvoker.java	2009-05-19 09:24:43 UTC (rev 26608)
@@ -535,7 +535,9 @@
             try {
                 courier = getCourier(targetEPR);
             } catch (CourierException e) {
-                logger.debug("Courier lookup failed for EPR [" + targetEPR + "] for Service [" + service + "] and Message ["+message.getHeader()+"].", e);
+                if (logger.isDebugEnabled()) {
+                    logger.debug("Courier lookup failed for EPR [" + targetEPR + "] for Service [" + service + "] and Message ["+message.getHeader()+"].", e);
+                }
             } catch (MalformedEPRException e) {
             	// probably an ESB-unaware EPR in the registry!!
 
@@ -562,7 +564,9 @@
                         replyToEPR = (currentReplyToEpr == null ? getReplyToAddress(targetEPR) : currentReplyToEpr) ;
 
                         if (replyToEPR == null) {
-                            logger.debug("Not using epr [" + targetEPR + "] for Service [" + service + "] and Message ["+message.getHeader()+"]. No reply-to address available for synchronous response.");
+                            if (logger.isDebugEnabled()) {
+                                logger.debug("Not using epr [" + targetEPR + "] for Service [" + service + "] and Message ["+message.getHeader()+"]. No reply-to address available for synchronous response.");
+                            }
                             return null;
                         }
                         call.setReplyTo(replyToEPR);
@@ -587,7 +591,9 @@
                 } catch (final CourierServiceBindException e) {
                     // meant to be masked by the SI fail-over
 
-                    logger.debug("Caught service lookup exception for EPR [" + targetEPR + "] and Service [" + service + "] and Message ["+message.getHeader()+"]. " + e.getMessage());
+                    if (logger.isDebugEnabled()) {
+                        logger.debug("Caught service lookup exception for EPR [" + targetEPR + "] and Service [" + service + "] and Message ["+message.getHeader()+"]. ", e);
+                    }
 
                     // could be stale EPR, so move on to next entry in registry.
                 } catch (final CourierMarshalUnmarshalException e) {
@@ -597,7 +603,9 @@
                 } catch (final CourierTransportException e) {
                     // meant to be masked by the SI fail-over
 
-                    logger.debug("Courier indicated transport related error "+e+" during send/receive with EPR [" + targetEPR + "] for Service [" + service + "] and Message ["+message.getHeader()+"]. " + e.getMessage());
+                    if (logger.isDebugEnabled()) {
+                        logger.debug("Courier indicated transport related error "+e+" during send/receive with EPR [" + targetEPR + "] for Service [" + service + "] and Message ["+message.getHeader()+"]. ", e);
+                    }
                 } catch (CourierException e) {
                     // probable config error. Log it and move on to next EPR/service entry.