Definitely the load to the database should be prevented. Will look into your fix once available. Thanks!
Jaakko On Tue, Jul 21, 2009 at 3:48 PM, Ciaran Gultnieks <[email protected]>wrote: > Jaakko Rajaniemi wrote: > > DBQueueManager for JabberQueueHandler can hang on some cases. I'm > quite sure > > that the line 58 ( > > > http://gitorious.org/laconica/mainline/blobs/master/lib/dbqueuemanager.php > ) > > causes the problem. The line is: > > > > $notice = $this->_nextItem($queue, null); > > Seems to me it's a bug. Even when it doesn't hang, it sucks up 100% > CPU and puts a massive load on the database. I am running with some > modifications that solve both problems, which I will commit when I've > tested for a bit longer. > > > I can think of at least two ways to fix this: > > 1. replace the null with some timeout > > 2. change _netItem function to service XMPP buffer > > The first is partially the answer - it should definitely have the > timeout. It also needs a sleep within the loop within nextItem though, > to prevent the heavy (and pointless) CPU and database usage. > > Cheers, > > Ciaran > -- Jaakko Sent from Helsinki, ES, Finland
_______________________________________________ Laconica-dev mailing list [email protected] http://mail.laconi.ca/mailman/listinfo/laconica-dev
