[asterisk-dev] [Code Review] 3559: sqlite3: Add ability to automatically retry query to busy database

Matt Jordan reviewboard at asterisk.org
Wed Jun 11 09:08:48 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3559/#review12125
-----------------------------------------------------------

Ship it!


For the non-AstDB parts, please go ahead and apply those to 1.8.

Thanks!

- Matt Jordan


On June 11, 2014, 5:56 a.m., Igor Goncharovsky wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3559/
> -----------------------------------------------------------
> 
> (Updated June 11, 2014, 5:56 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23766
>     https://issues.asterisk.org/jira/browse/ASTERISK-23766
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> We have faced situation when using CDR and CEL by sqlite3 modules. With system having high load (~100 concurrent calls created by sipp) we found many cdr and cel records missed. There is special finction in sqlite3, that make able to fix this situation - sqlite3_wait_timeout, that also can replace awful code cdr_sqlite3 ad cel_sqlite3 modules. Also this function can be used for aastdb and res_config_sqlite3 to avoid missed writes to sqlite db.
> 
> 
> Diffs
> -----
> 
>   /trunk/res/res_config_sqlite3.c 414674 
>   /trunk/main/db.c 414674 
>   /trunk/cel/cel_sqlite3_custom.c 414674 
>   /trunk/cdr/cdr_sqlite3_custom.c 414674 
> 
> Diff: https://reviewboard.asterisk.org/r/3559/diff/
> 
> 
> Testing
> -------
> 
> Adding sqlite3_wait_timeout already used as patch in AskoziaPBX and show good result.
> 
> 
> Thanks,
> 
> Igor Goncharovsky
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140611/ce7b4847/attachment.html>


More information about the asterisk-dev mailing list