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

[Kabir Khan]

Thanks,

That was next on my list, and I'll try to fix it tomorrow. Do the  
tests create a report of the leaks somewhere? Does jboss profiler work  
on OS X?

On 1 Apr 2009, at 20:02, Brian Stansberry wrote:

> 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
> _______________________________________________
> jboss-development mailing list
> jboss-development at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-development