https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6942

Henrik Krohns <h...@hege.li> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |h...@hege.li

--- Comment #4 from Henrik Krohns <h...@hege.li> ---
Gratitude for improving my quick hack. :-)

A small bug in your change: if bayes_auto_expire is not set, connection crashed
due to Redis receiving expire command with undefined value.

Sending        Redis.pm
Transmitting file data .
Committed revision 1488285.

To discuss further, I'm not sure if it makes sense to allow Redis without
expire. Most people have bayes_auto_expire disabled, it requires careful
reading of docs to figure out you actually need it so db won't infinitely
grow.. and it's too late to change after without rewriting db..

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to