Re: [jboss-dev] Re: AOP 2.1.0 in AS 5?

I've been looking into the cause of the classloader leak test failures[1] and am seeing leaks leading through org .jboss .aop.asintegration.jboss5.RegisterModuleCallback.registeredModules Looking further into it, I see RegisterModuleCallback.registerModule(Module) is called, but nowhere is removeModule(Module) called. The registerModule call is from JBossClDelegatingClassPoolFactory.create(...). [1] http://hudson.qa.jboss.com/hudson/view/JBoss%20AS/job/JBoss-AS-5.1.x-test... Kabir Khan wrote: > I've released AOP 2.1.0.CR2 and upgraded AS Branch_5_x to use that > On 1 Apr 2009, at 18:38, Jason T. Greene wrote: >> Great thanks. >> >> Kabir Khan wrote: >>> It seems the classloader used to deploy the tx, security etc. >>> aspects is wrong so it cannot find the base-aspects.xml file >>> before running these tests. I'll fix and deploy a new CR. >>> On 1 Apr 2009, at 11:13, Kabir Khan wrote: >>>> I tested these when I commited, and reverting my local copy to >>>> my last commit (revision 86202) the aop tests all work. It must >>>> be something commited later. The main problems are anything >>>> involving the transaction aspects (ScopedUnitTestCase, >>>> ScopedAttachUnitTestCase, TxLockTestCase, TxTestCase, >>>> VersionedObjectTestCase) and security aspects >>>> (SecurityUnitTestCase). >>>> >>>> Looking at the changes to thirdparty jars nothing springs out >>>> http://fisheye.jboss.com/browse/~br=Branch_5_x/JBossAS/branches/ >>>> Branch_5_x/component-matrix/pom.xml?r1=86546&r2=86201 >>>> >>>> >>>> >>>> On 31 Mar 2009, at 12:20, Kabir Khan wrote: >>>> >>>>> Sure >>>>> On 30 Mar 2009, at 21:30, Jason T. Greene wrote: >>>>> >>>>>> Kabir Khan wrote: >>>>>>> I have upgraded Branch_5_x to use AOP 2.1.0.CR1 and switched >>>>>>> to use the new classpools that understand AS classloading >>>>>>> better. If there are no problems I'll do a GA over the next >>>>>>> week or so >>>>>> >>>>>> It looks like a number of AOP tests are now failing after this >>>>>> update. Can you look into these? >>>>>> >>>>>> http://hudson.qa.jboss.com/hudson/view/JBoss%20AS/job/JBoss-AS-5.1.x-test... >>>>>> >>>>>> -- >>>>>> Jason T. Greene >>>>>> JBoss, a division of Red Hat >>>>> >>>>> _______________________________________________ >>>>> jboss-development mailing list >>>>> jboss-development(a)lists.jboss.org >>>>> https://lists.jboss.org/mailman/listinfo/jboss-development >>>> >>>> _______________________________________________ >>>> jboss-development mailing list >>>> jboss-development(a)lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/jboss-development >> >> >> -- >> Jason T. Greene >> JBoss, a division of Red Hat > _______________________________________________ > jboss-development mailing list > jboss-development(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/jboss-development -- Brian Stansberry Lead, AS Clustering JBoss, a division of Red Hat brian.stansberry(a)redhat.com _______________________________________________ jboss-development mailing list jboss-development(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/jboss-development