Re: [jboss-as7-dev] logger config per deployment / app

You have a bug in ConfigFilter, it also picks up directories. Recursing is already taken care by the VFS code, the filter should just filter out actual leaves / files. https://github.com/jbossas/jboss-as/pull/2387 > Yes it's on by default, but you can turn it off by adding a system > property org.jboss.as.logging.per-deployment and setting it to false. > > It shouldn't be that slow. I suppose we could limit the search to > specific directories, but from my tests it hasn't been too slow. > > On 05/25/2012 02:38 AM, Ales Justin wrote: >> Is this on by default? >> >> LoggingConfigurationProcessor >> >> private VirtualFile findConfigFile(ResourceRoot resourceRoot) >> throws DeploymentUnitProcessingException { >> VirtualFile result = null; >> try { >> final List<VirtualFile> configFiles = >> resourceRoot.getRoot().getChildrenRecursively(ConfigFilter.INSTANCE); >> for (final VirtualFile file : configFiles) { >> >> As this looks slow ... >> >> On May 25, 2012, at 12:13 AM, James Perkins wrote: >> >>> Only if you add a logging.properties file to your deployment. Once you >>> do that though, you can no longer configure it via the management >>> console. >>> >>> I was working on a concept of logging profiles that could be >>> assigned to >>> deployments. Though that would still require a change to the >>> deployment. >>> It could be interesting to isolate all deployment logging >>> on-demand. Not >>> quite sure how it would work, but I could see it being useful for >>> debugging. >>> >>> On 05/24/2012 03:00 PM, Ales Justin wrote: >>>> Can you already configure loggers per deployment? >>>> e.g. I want org.hibernate.search at trace for my app, but not for >>>> the whole AS >>>> >>>> If yes, how? >>>> >>>> I mean, is there a better way than adding the old TCCL filter hack >>>> I once helped to write. :) >>>> >>>> >>>> _______________________________________________ >>>> jboss-as7-dev mailing list >>>> jboss-as7-dev(a)lists.jboss.org <mailto:jboss-as7-dev@lists.jboss.org> >>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev >>> -- >>> James R. Perkins >>> JBoss by Red Hat >>> >>> _______________________________________________ >>> jboss-as7-dev mailing list >>> jboss-as7-dev(a)lists.jboss.org <mailto:jboss-as7-dev@lists.jboss.org> >>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev >> >> _______________________________________________ >> jboss-as7-dev mailing list >> jboss-as7-dev(a)lists.jboss.org <mailto:jboss-as7-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev > > -- > James R. Perkins > JBoss by Red Hat > > _______________________________________________ > jboss-as7-dev mailing list > jboss-as7-dev(a)lists.jboss.org <mailto:jboss-as7-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/jboss-as7-dev _______________________________________________ jboss-as7-dev mailing list jboss-as7-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/jboss-as7-dev