[jboss-cvs] JBossAS SVN: r103638 - trunk/ejb3/src/main/java/org/jboss/ejb3/deployers.

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

Author: david.lloyd at jboss.com
Date: 2010-04-07 12:01:26 -0400 (Wed, 07 Apr 2010)
New Revision: 103638

Modified:
   trunk/ejb3/src/main/java/org/jboss/ejb3/deployers/JBossASKernel.java
Log:
JBAS-7640: Chill the logging a bit

Modified: trunk/ejb3/src/main/java/org/jboss/ejb3/deployers/JBossASKernel.java
===================================================================
--- trunk/ejb3/src/main/java/org/jboss/ejb3/deployers/JBossASKernel.java	2010-04-07 15:30:42 UTC (rev 103637)
+++ trunk/ejb3/src/main/java/org/jboss/ejb3/deployers/JBossASKernel.java	2010-04-07 16:01:26 UTC (rev 103638)
@@ -104,7 +104,7 @@
             String kdname = unit.getShortName();
             deployment.setName(kdname);
             unit.addAttachment(KernelDeployment.class.getName(), deployment);
-            log.info("Created KernelDeployment for: "+unit.getShortName());
+            log.debugf("Created KernelDeployment for: %s", unit.getShortName());
          }
       }
 
@@ -115,27 +115,29 @@
       bean.setDepends(policy.getDependencies());
       bean.setDemands(policy.getDemands());
       bean.setSupplies(policy.getSupplies());
-      log.info("installing bean: " + name);
-      log.info("  with dependencies:");
-      for (Object obj : policy.getDependencies())
-      {
-         Object msgObject = obj;
-         if (obj instanceof AbstractDemandMetaData)
+      if (log.isTraceEnabled()) {
+         log.tracef("installing bean: %s", name);
+         log.trace("  with dependencies:");
+         for (Object obj : policy.getDependencies())
          {
-            msgObject = ((AbstractDemandMetaData)obj).getDemand();
+            Object msgObject = obj;
+            if (obj instanceof AbstractDemandMetaData)
+            {
+               msgObject = ((AbstractDemandMetaData)obj).getDemand();
+            }
+            log.tracef("\t%s", msgObject);
          }
-         log.info("\t" + msgObject);
+         log.trace("  and demands:");
+         for(DemandMetaData dmd : policy.getDemands())
+         {
+            log.tracef("\t%s; Required: %s", dmd.getDemand(), dmd.getWhenRequired().getStateString());
+         }
+         log.trace("  and supplies:");
+         for(SupplyMetaData smd : policy.getSupplies())
+         {
+            log.tracef("\t%s", smd.getSupply());
+         }
       }
-      log.info("  and demands:");
-      for(DemandMetaData dmd : policy.getDemands())
-      {
-         log.info("\t" + dmd.getDemand() + "; Required: " + dmd.getWhenRequired().getStateString());
-      }
-      log.info("  and supplies:");
-      for(SupplyMetaData smd : policy.getSupplies())
-      {
-         log.info("\t" + smd.getSupply());
-      }
 
       if(service instanceof EJBContainer)
       {
@@ -164,7 +166,7 @@
       {
          // Just add the mc bean metadata to the unit
          deployment.getBeanFactories().add(bean);
-         log.info("Added bean("+name+") to KernelDeployment of: "+unit.getShortName());
+         log.debugf("Added bean(%s) to KernelDeployment of: %s", name, unit.getShortName());
       }
       else
       {
@@ -177,7 +179,7 @@
             }
             catch (IllegalStateException e){}
                   
-            log.info("Installing bean("+name+") into kernel");
+            log.debugf("Installing bean(%s) into kernel", name);
             kernel.getController().install(bean);
          }
          catch (Throwable throwable)