I just added a super-simple test case for the include tag to the core test suite. Include is a special kind of tag, so hopefully it's the only one leaking. Once we fix this problem, a retest will definitely be in order.
-----Original Message----- From: Brett Porter [mailto:[EMAIL PROTECTED] Sent: Monday, January 24, 2005 9:47 PM To: Jakarta Commons Developers List Subject: RE: [jelly] Maven JSL memory leak and Jelly Yup. AFAICT, it's been there since it's creation. Do you need some additional info from me? I'm happy to try to make a smaller test case and/or test with Maven later if required. - Brett Quoting Hans Gilde <[EMAIL PROTECTED]>: > There's definitely a leak in the include tag. > > Brett, I'm guessing that this leak also exists pre-RC2, right? > > -----Original Message----- > From: Brett Porter [mailto:[EMAIL PROTECTED] > Sent: Monday, January 24, 2005 7:30 AM > To: Jakarta Commons Developers List > Subject: Re: [jelly] Maven JSL memory leak and Jelly > > Paul Libbrecht wrote: > > > Ideally, a test-case would be awesome, even if it refers to some > > far-away stylesheet... > > I'll try and narrow it down first by cutting down site.jsl to the > minimum that leaks. > > > > > Can you try calling .clear() on the result of this > > context.runScript(uri, output, isExport(), isInherit()) > > (and the other call). > > so try with context.clear() inside IncludeTag? Ok, will try tomorrow. > > > > > Maybe that'll help. > > In all cases, this context is gc-ed shortly after, I believe... so I > > see no reasons for big leaks at the tag-cache level. > > It leaks 200k every time site.jsl is run, so should be easy to find, > right? :) > > > > > Also, maybe it would help to give more details where to go... I think > > this was reported about very long ago so maybe a distribution > > maven-1.0.2 or such should have this bug ? > > easiest to run it against Maven's trunk really as it is compatible with > Jelly RC2 (RC1 was broken, so it currently uses beta-4 until RC2 is > released). RC2 won't work with Maven 1.0.x. > > If you want to give it a go: > - check out Maven from SVN and build: > svn co http://svn.apache.org/repos/asf/maven/maven-1/core/trunk > maven-1/core/trunk > svn co http://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk > maven-1/plugins/trunk > - build maven inside maven-1/core/trunk (see > http://www.apache.org/~brett/maven-stage-site/developers/building-from-sourc > e.html, > make sure you reassign a new MAVEN_HOME!) > - run "maven xdoc" and watch memory go... > - twiddle xdoc plugin and run "maven plugin:install" to test. Use the > memory tag lib I just added to jelly to test. <memory:gc/> and > <memory:showFree />. > > Hopefully I'll have more information tomorrow, I'm going to bed now. I'm > keen to kill this before Jelly 1.0 if possible. > > Thanks, > Brett > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
