On Sun, Feb 8, 2009 at 5:36 AM, Marius <marius.dan...@gmail.com> wrote:
> > Related with the above ... would it be ok with you to have a comet > long-polling interval configurable from somewhere (i.e LiftRules)? ... > currently it is hardcoded to 10 seconds. The Comet long poll should be restarted 100ms after the last successful long poll. This should not be configurable. If you don't want long poll, don't put Comet components on the page. I'm cool with configurability of long poll failure timeout. > > > On Feb 8, 10: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. > > > > 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. > > > > 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? > > > > 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.netandthere > 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 -~----------~----~----~----~------~----~------~--~---
