https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6942
--- Comment #5 from Mark Martinec <mark.marti...@ijs.si> --- > 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 > Committed revision 1488285. Thanks for noticing and a fix! > 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.. The only reason I think that bayes_auto_expire should also be obeyed by Redis is to go with the current documentation on bayes_auto_expire, i.e. not introducing arbitrary/unnecessary exceptions, the POLA principle. -- You are receiving this mail because: You are the assignee for the bug.
