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

[Brian Stansberry]

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-testSuite-sun15/169/testReport/org.jboss.test.classloader.leak.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-testSuite-sun15/lastBuild/testReport/ 
>>>>>>
>>>>>>
>>>>>> -- 
>>>>>> Jason T. Greene
>>>>>> JBoss, a division of Red Hat
>>>>>
>>>>> _______________________________________________
>>>>> jboss-development mailing list
>>>>> jboss-development at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-development
>>>>
>>>> _______________________________________________
>>>> jboss-development mailing list
>>>> jboss-development at 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 at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-development


-- 
Brian Stansberry
Lead, AS Clustering
JBoss, a division of Red Hat
brian.stansberry at redhat.com