http://bugzilla.spamassassin.org/show_bug.cgi?id=3334 ------- Additional Comments From parkerm@pobox.com 2004-06-04 18:43 -------
Subject: Re: journal_live_path returning an invalid path for bayes _journal
On Fri, Jun 04, 2004 at 06:19:35PM -0700, bugzilla-daemon@bugzilla.spamassassin.org wrote:
>
> ------- Additional Comments From jm@jmason.org 2004-06-04 18:19 -------
> Mike -- maintain a copy of what the $conf member was, at the last time the
> sed_path() lookup was performed. if that changes, then the cached value is
> invalidated.
>
We do. The problem is, and I haven't traced it all the way through so
I might be a little off, that when spamd starts up it does the little
pre-compile step that sets up temp userstate dirs and what not and
processes a fake msg. Then it sets things back and is ready for
processing. The DBM module caches the journal path/filename with this
temp userstate and just returns that each time.
It's ok to cache because we generally create a new object each time we
change users. However this startup step doesn't create a new object
so we see the problem.
My current fix, is during the startup (compile_now) step it calls a
clear_cached_dirs method on the Bayes module that clears any cached
values (currently just this one) and we're all good.
Not employing a cache causes an ~8% slow down in processing.
Michael
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.