https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8331

--- Comment #4 from Giovanni Bechis <giova...@paclan.it> ---
(In reply to Jan Smutný from comment #3)
> From my point of view, using "bayes_sql_dsn" in case of Redis backend is
> confusing and my preference would be to don't use it in case of Redis
> backend at all. The only reason I'm keeping it is for backward compatibility.
> So my preference is using "bayes_redis_*" explicit options for this new
> functionality.

I do not disagree with that, but if we want to go that path, bayes_redis_*
should be documented in Mail::SpamAssassin::Conf the same way bayes_sql_* is
documented and contrib/HOWTO.Bayes-Redis/SA-rule/bayes_redis.cf should be
updated accordingly.

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

Reply via email to