commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject RE: [jelly] Maven JSL memory leak and Jelly
Date Tue, 25 Jan 2005 03:09:20 GMT
Thanks Hans!

That's what I was thinking of doing, so I'm glad you've got it under control.

When I was testing, the size of the included JSL affected the amount of memory
leaked, so I guess it is tag caching again?

I tried a context.clear[Thread]ScriptData after the execution of runScript in
include, but that didn't help. Unfortunately that's all I had time for this
morning before work.

Good luck...

Cheers,
Brett

Quoting Hans Gilde <hgilde-commons@earthlink.net>:

> 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:brett@apache.org] 
> 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 <hgilde-commons@earthlink.net>:
> 
> > 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:brett@apache.org] 
> > 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: commons-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> > 
> > 
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> > 
> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 




---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message