Bugs item #1658305, was opened at 2007-02-12 19:07 Message generated for change (Comment added) made by anca_vamanu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1658305&group_id=139143
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: ver devel >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Aron Rosenberg (amr42) >Assigned to: anca (anca_vamanu) Summary: SQL Indexes on presence.active_watchers table Initial Comment: >From our preformance testing of the presence module there should be an index >(not unique) on the active_watchers table covering (to_domain,to_user,to_event) This index would handle the very frequent query when a NOTIFY needs to be sent out from notify.c in lines 286 and 735. Without this index, a full table scan is performed any time a new PUBLISH is received and you are looking up which users need to get a NOTIFY The attached file is a 'rough' patch to add the index to the proper locations. The postgres part is completely untested since we use mysql. ---------------------------------------------------------------------- >Comment By: anca (anca_vamanu) Date: 2007-02-13 14:27 Message: Logged In: YES user_id=1614776 Originator: NO Hello, Thank you for the patches. I have applied them and commited the changes. Best regards, Anca Vamanu ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1658305&group_id=139143 _______________________________________________ Devel mailing list [email protected] http://openser.org/cgi-bin/mailman/listinfo/devel
