[jboss-dev] Re: [jboss-cvs] JBossAS SVN: r71836 - projects/jboss-deployers/trunk/deployers-vfs/src/main/org/jboss/deployers/vfs/deployer/kernel.

Adrian Brock abrock at redhat.com
Wed Apr 9 09:43:36 EDT 2008


We want to be trying reduce the noise in the logging not increasing
it. :-)

This is just duplicate logging see AbstractDeploymentContext:

public void addComponent(DeploymentContext component)
{
   if (component == null)
      throw new IllegalArgumentException("Null component");
   deployed();
   components.add(component);
   log.debug("Added component " + component.getName() + " to " +
getName());
}

On Wed, 2008-04-09 at 09:19 -0400, jboss-cvs-commits at lists.jboss.org
wrote:
> Author: scott.stark at jboss.org
> Date: 2008-04-09 09:19:41 -0400 (Wed, 09 Apr 2008)
> New Revision: 71836
> 
> Modified:
>    projects/jboss-deployers/trunk/deployers-vfs/src/main/org/jboss/deployers/vfs/deployer/kernel/KernelDeploymentDeployer.java
> Log:
> Log the addition of bean components
> 
> Modified: projects/jboss-deployers/trunk/deployers-vfs/src/main/org/jboss/deployers/vfs/deployer/kernel/KernelDeploymentDeployer.java
> ===================================================================
> --- projects/jboss-deployers/trunk/deployers-vfs/src/main/org/jboss/deployers/vfs/deployer/kernel/KernelDeploymentDeployer.java	2008-04-09 12:57:52 UTC (rev 71835)
> +++ projects/jboss-deployers/trunk/deployers-vfs/src/main/org/jboss/deployers/vfs/deployer/kernel/KernelDeploymentDeployer.java	2008-04-09 13:19:41 UTC (rev 71836)
> @@ -29,9 +29,11 @@
>  import org.jboss.deployers.spi.deployer.helpers.DeploymentVisitor;
>  import org.jboss.deployers.structure.spi.DeploymentUnit;
>  import org.jboss.kernel.spi.deployment.KernelDeployment;
> +import org.jboss.logging.Logger;
>  
>  public class KernelDeploymentDeployer extends AbstractComponentDeployer<KernelDeployment, BeanMetaData>
>  {
> +   private static Logger log = Logger.getLogger(KernelDeploymentDeployer.class);
>     /**
>      * Create a new KernelDeploymentDeployer.
>      */
> @@ -45,6 +47,7 @@
>     {
>        DeploymentUnit component = unit.addComponent(bean.getName());
>        component.addAttachment(BeanMetaData.class.getName(), bean);
> +      log.debug("addBeanComponent, unit="+unit.getSimpleName()+", bean="+bean.getName());
>     }
>  
>     protected static void removeBeanComponent(DeploymentUnit unit, BeanMetaData bean)
> 
> _______________________________________________
> jboss-cvs-commits mailing list
> jboss-cvs-commits at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-cvs-commits
-- 
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Adrian Brock
Chief Scientist
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx




More information about the jboss-development mailing list