Summary: | ASTERISK-11125: [patch] Additional AMD logging and added maximum word length option. | ||
Reporter: | David Van Ginneken (davevg) | Labels: | |
Date Opened: | 2007-12-28 09:31:15.000-0600 | Date Closed: | 2007-12-28 10:08:40.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_amd |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) app_amd-trunk.diff | |
Description: | This patch adds additional logging to the output to make it easier to tune AMD. It is a port to trunk of a patch we are running on multiple servers in a client location using 1.4. List of changes: The channel name has been added to all verbose output. A new option maximumWordLength has been added. It will log the duration of words that do not meet the minimum word duration. If we received silence before a greeting, it will log the silence duration. Only consider us in the greeting if we actually detected a valid word duration. | ||
Comments: | By: Mark Michelson (mmichelson) 2007-12-28 10:03:11.000-0600 Patch looks fine by me and it's being used in production, so it sounds like it's good to go. I'm just going to make a note in the CHANGES file and then I'll get this committed. Thanks for the patch! By: Digium Subversion (svnbot) 2007-12-28 10:08:40.000-0600 Repository: asterisk Revision: 95167 U trunk/CHANGES U trunk/apps/app_amd.c ------------------------------------------------------------------------ r95167 | mmichelson | 2007-12-28 10:08:38 -0600 (Fri, 28 Dec 2007) | 11 lines Some changes to app_amd. The channel name is printed in verbose messages maximumWordLength option added. Duration of words that do not meet the minimum word duration will be logged The duration of pre-greeting silence will be logged Only consider us in the greeting if we actually detected a valid word duration. (closes issue ASTERISK-11125, reported and patched by davevg) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=95167 |