Re: Sqlbox performance patch

From a quick overview, I found 2 typos.
You should replace &save_mo with &save_mt and &save_dlr respectively in:

+    if (cfg_get_bool(&save_mo, grp, octstr_imm("save-mt")) == -1)
+        save_mt = 1;
+
+    if (cfg_get_bool(&save_mo, grp, octstr_imm("save-dlr")) == -1)
+        save_dlr = 1;
+

Br, Rinor

On 08/11/2013 03:01 PM, Rene Kluwen wrote:
>
> Attached is a patch (adapted from the one of Alejandro Guerrieri)
> which adds the following features to sqlbox, patched against current
> sqlbox svn trunk (revision 52).
>
>  
>
> 1.       Configuration parameter: limit-per-cycle, which processes a
> number of sql rows at a time.
>
> 2.       Configuration parameters: save-dlr, save-mo, save-mt which
> indicate whether to save particular messages in the sql-log-table.
>
>  
>
> Also, a few other performance related improvements were made, such as
> deleting all messages in the cycle at once (one DELETE statement) as
> well as inserting those messages in one time as well.
>
> All the improvements only apply to the MySQL driver. Other database
> back ends work like usual.
>
> More information in the documentation that also is included in the patch.
>
>  
>
> I would appreciate it if somebody could test this patch before
> committing it to the trunk.
>
>  
>
> == Rene
>
>  
>