On Sun, Feb 8, 2009 at 12:37 AM, Marius <marius.dan...@gmail.com> wrote:
> > Dave this is way awesome! > > I have some questions/suggestions if I may > > > 1. Currently Lift purges the unseen functions that are older then 10 > minutes. I think it might help to make this time window configurable > via LiftRules so that people may adjust it to fit their specific > application needs. Sure. > > > 2. AddLiftGCToBody should probably not renders anything related with > GC if the GC nodes list is empty? ... or have the ability to disable > the GC support which might be quite handy for mobile web applications > used for limited browsers. > Disable GC... cool. Not sending the Ajax request if the list is empty, cool. But it's possible that items come onto the page via Ajax or Comet. Thus, we have to run the node walk as long as GC is enabled. > > 3. Lift for every page is starting the JS GC "daemon" that > periodically sends the GC request up (every 75 seconds for success or > 15 seconds for failure). Shouldn't we make these these times intervals > configurable from LiftRules? > Sure. > > Thoughts? > > Br's, > Marius > > On Feb 8, 2:05 am, David Pollak <feeder.of.the.be...@gmail.com> wrote: > > Folks, > > > > I've found and squashed a bunch of memory retention issues in the Lift > comet > > stuff (and added the garbage collection for functions.) > > > > I've tested the fixes against the livehttp://demo.liftweb.netand there > has > > been no memory growth (except in Derby, but that's not Lift's problem) > for > > 24 hours. > > > > I've been using YouKit (http://yourkit.com/) to profile that > application. > > I've gotta say that YourKit is awesome, even for debugging remote code. > I'd > > like to thank YourKit again for contributing licenses to the Lift > > committers. > > > > I've dialed the heap size for demo.liftweb.net from 1GB to 192M. In > > practice, the actual heap size for the site never grew beyond about 10MB. > > > > Thanks, > > > > David > > > > PS -- Yes, my confidence about making 1.0 on 2/26 is very high. > > > > -- > > Lift, the simply functional web frameworkhttp://liftweb.net > > Beginning Scalahttp://www.apress.com/book/view/1430219890 > > Follow me:http://twitter.com/dpp > > Git some:http://github.com/dpp > > > -- Lift, the simply functional web framework http://liftweb.net Beginning Scala http://www.apress.com/book/view/1430219890 Follow me: http://twitter.com/dpp Git some: http://github.com/dpp --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Lift" group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~----------~----~----~----~------~----~------~--~---
