From: Eric Liang
I guess the designment is to get a static element container, but what about if there are a great amount users registered while just a few of them are active? Meanwhile, the empty elements list will come to a bottleneck while getting an element not exists in the hash table.

Seems correct.
Did you do any benchmarking?

two patches are attached, while the second one is to print the stat of xhash, just for debug the initial size( sm->users/ sm->sessions for example).

Cool. Have you tested it under live conditions (with real users)?

Could you create a new Bug on Launchpad https://bugs.launchpad.net/jabberd2/+filebug and attach the patches to it?


--
Tomasz Sterna
Instant Messaging & EDI Consultant
Open Source Developer
http://tomasz.sterna.tv/ http://www.xiaoka.com/

--
To unsubscribe send a mail to [email protected]

Reply via email to