[Devel] [ openser-Bugs-1658305 ] SQL Indexes on presence.active_watchers table

[SourceForge.net]

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