| Summary: | ASTERISK-06825: [patch] Timewaste on count messages in app_voicemail with odbc support | ||
| Reporter: | dlu (dlu) | Labels: | |
| Date Opened: | 2006-04-21 16:34:12 | Date Closed: | 2006-05-09 00:56:14 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Applications/app_voicemail |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) app_voicemail.patch | |
| Description: | I have installed the app_voicemail with odbc support an we have many accounts on a * box. Even a couple functions fire messagecount for voicemail. The tablestructure has a index on dir but is never used because a "like" statment dont use indexes. Also every count is a sequential search on the entire vocemessagetable. This bug is not minor but my fix tweak the system and save resources :-) ****** ADDITIONAL INFORMATION ****** It would be nice to use a variable like vmbasedir=/var/spool/astereisk/voicemail or something to break the hardcoding in my patch. | ||
| Comments: | By: Jason Parker (jparker) 2006-04-22 00:07:24 I disagree with this method. I think we should add a new column, ala mailboxcontext and mailboxuser, or whatever they're called. If we add a new folder column, this problem would be solved completely. Maybe now would also be a good time to remove the ODBC_STORAGE_EXTENDED define, since 1.4 will be coming soon. By: Olle Johansson (oej) 2006-04-23 01:25:22 ...and how is this related to SIP? (Check category when opening an issue report, please) By: Clod Patry (junky) 2006-04-23 13:06:59 could u use the constant VM_SPOOL_DIR instead of hard-coding paths? Thanks. By: Tilghman Lesher (tilghman) 2006-05-09 00:56:14 Committed to trunk. | ||