[Home]

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-0600Date Closed:2007-12-28 10:08:40.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents: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