noon
Author: alessio.soldano(a)jboss.com
Date: 2009-09-04 13:00:53 -0400 (Fri, 04 Sep 2009)
New Revision: 10655
Modified:
framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java
Log:
[JBWS-2750] skip debug logs when debug level is not enabled
Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java
===================================================================
---
framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java 2009-09-04
17:00:03 UTC (rev 10654)
+++
framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java 2009-09-04
17:00:53 UTC (rev 10655)
@@ -67,7 +67,9 @@
*/
public <T> T getSPI(Class<T> spiType)
{
- log.debug("provide SPI '" + spiType + "'");
+ boolean debugEnabled = log.isDebugEnabled();
+ if (debugEnabled)
+ log.debug("provide SPI '" + spiType + "'");
T returnType = null;
@@ -149,7 +151,8 @@
throw new WSFException("Failed to provide SPI '" + spiType +
"'");
// help debugging
- log.debug(spiType + " Implementation: " + returnType);
+ if (debugEnabled)
+ log.debug(spiType + " Implementation: " + returnType);
return returnType;
}
Modified:
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java
===================================================================
---
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java 2009-09-04
17:00:03 UTC (rev 10654)
+++
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java 2009-09-04
17:00:53 UTC (rev 10655)
@@ -114,7 +114,8 @@
}
else
{
- log.debug("Cannot get " + resourcePath + " from root
file, trying with additional metadata files", e);
+ if (log.isDebugEnabled())
+ log.debug("Cannot get " + resourcePath + " from root
file, trying with additional metadata files", e);
}
}
@@ -142,7 +143,8 @@
}
catch (IOException e)
{
- log.debug("Cannot get " + resourcePath + " from
" + uvf, e);
+ if (log.isDebugEnabled())
+ log.debug("Cannot get " + resourcePath + " from
" + uvf, e);
}
}
}
Modified:
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java
===================================================================
---
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java 2009-09-04
17:00:03 UTC (rev 10654)
+++
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java 2009-09-04
17:00:53 UTC (rev 10655)
@@ -57,7 +57,8 @@
public DeploymentAspectManager getDeploymentAspectManager(String beanName)
{
- log.debug("DeploymentAspectManager for: " + beanName);
+ if (log.isDebugEnabled())
+ log.debug("DeploymentAspectManager for: " + beanName);
return new KernelAwareSPIFactory().getKernelProvidedSPI(beanName,
DeploymentAspectManager.class);
}
}
Modified:
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java
===================================================================
---
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java 2009-09-04
17:00:03 UTC (rev 10654)
+++
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java 2009-09-04
17:00:53 UTC (rev 10655)
@@ -40,7 +40,8 @@
public void start(Endpoint ep)
{
- log.debug("Start: " + ep.getName());
+ if (log.isDebugEnabled())
+ log.debug("Start: " + ep.getName());
EndpointState state = ep.getState();
if (state != EndpointState.UNDEFINED)
@@ -63,7 +64,8 @@
public void stop(Endpoint ep)
{
- log.debug("Stop: " + ep.getName());
+ if (log.isDebugEnabled())
+ log.debug("Stop: " + ep.getName());
EndpointState state = ep.getState();
if (state != EndpointState.STARTED)
Modified:
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java
===================================================================
---
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java 2009-09-04
17:00:03 UTC (rev 10654)
+++
framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java 2009-09-04
17:00:53 UTC (rev 10655)
@@ -78,12 +78,15 @@
depAspects.clear();
depAspects.addAll(aspects);
- // Debug the aspect list
- StringBuilder builder = new StringBuilder("setDeploymentAspects on " +
name);
- for (DeploymentAspect aspect : aspects)
- builder.append("\n " + aspect.getClass().getName() + " provides:
" + aspect.getProvidesAsSet() + " requires: " +
aspect.getRequiresAsSet());
-
- log.debug(builder);
+ if (log.isDebugEnabled())
+ {
+ // Debug the aspect list
+ StringBuilder builder = new StringBuilder("setDeploymentAspects on " +
name);
+ for (DeploymentAspect aspect : aspects)
+ builder.append("\n " + aspect.getClass().getName() + "
provides: " + aspect.getProvidesAsSet() + " requires: " +
aspect.getRequiresAsSet());
+
+ log.debug(builder);
+ }
}
/**
@@ -165,7 +168,8 @@
{
String name = aspect.getClass().getName();
name = name.substring(name.lastIndexOf(".") + 1);
- log.debug("[" + this.name + "]" + name + ":" +
method);
+ if (log.isDebugEnabled())
+ log.debug("[" + this.name + "]" + name + ":" +
method);
}
}