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

--- Comment #6 from Henrik Krohns <[email protected]> 2012-04-24 09:08:38 UTC ---
(In reply to comment #5)
> (In reply to comment #4)
> > New patch has the same root issue as original sa-awl has - potentially 
> > stores a
> > lot of keys in memory.
> 
> From what I understand, the amount of deleted keys normally would be very 
> small
> compared to previous code which arrayed _all_ keys. So there really isn't an
> issue here?

Nothing against making it safe for the worst scenario though.. I would go for
KISS method and just restart when @deleted_keys is over 100k-1M keys. I doubt
anyone would encounter the limit in normal usage and it's pretty much 0-cpu
operation to check array size in the loop. If someone is deleting that much
keys, probably he's deleting most of them, thus restarting pretty much starts
where it was left at..

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to