Re: [Sqlgrey-users] [SPAM] Re: Patches for SQLgrey

[Michal L. <ml...@lo...>]

Michal Ludvig wrote:

> Eventually we could rework the whole cleaning process. Instead of doing
> (potentially massive) cleanups in fixed time periods there could be a
> minor cleanup on every, say, 1000th call. No need to count the calls,
> simple 'if(int(rand(1000))==0) { .. cleanup .. }' should be sufficient.

Actually... attached is a patch that implements it. I'm running it with
cleanup_chance=10 (ie 1:10 chance to perform cleanup) on one of my
light-loaded servers and it seems to do the job. So far I've seen 13
cleanups in 150 greylist checks which is quite on track.


BTW How about dropping the async cleanup path completely? Given the
scary comment in sqlgrey.conf I don't believe anyone dares to use it anyway.

Michal