[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: Serious flaw in ActionScheduler::moveMessage()
From:       Don Sanders <sanders () kde ! org>
Date:       2005-12-09 6:52:42
Message-ID: 200512091652.43101.sanders () kde ! org
[Download RAW message or body]

On Thursday 08 December 2005 06:43, Andreas Gungl wrote:> Hi, I think I've finally \
fixed the async filtering. I want to sum> up the fixes to keep you up-to-date:>> 1) \
Some parts were guarded by an "#if 0" but were needed to get the> threadweaver \
going.> 2) Changes to a message made in external filters were not> recognized in \
subsequent filters (filter rules).> 3) Modifications in external filters which made a \
message loosing> the X-KMail-Filtered header lead to a crash.>> As far as I have \
tested (and I did many tests in the last two days)> the async filtering works quite \
stable now. If you still encounter> strange effects, please let me know. This is \
great news. Thanks very much for caring. BTW I read your previous message about this \
bug depending on default sa behavior. Thanks for that info.
> I've ported the changes to trunk. However I wonder if it would make> sense to move \
> some of the actionscheduler functionality into> threadweaver jobs to make the \
> filtering of a message more atomic> (but still in the background).
(I think Till actually was thinking about dropping use of threadweaver).
Not sure what you mean by more atomic. The action scheduler creates a new message \
puts that into the destination folder and then removes the original message.  I guess \
you could end up with a duplicated message if the timing of a crash was just right. \
But I can't see how using threadweaver would solve that. There are two main \
outstanding concerns I have. 1) Sometimes imap messages aren't filtered successfully \
(due to timeouts or whatever) and are left in the inbox. So I would like to add some \
logic to check for these left over messages and do a second pass to filter them. 2) \
Several years ago we got bug reports about losing mail when KMail crashes. So I put \
in fsyncs that are applied everytime a message is added to a folder. This had the \
unfortunate side effect of making KMail more blocking during filtering.
> Perhaps we can discuss some issues> during the Osnabrück meeting.
(I don't think I'll be there.)
Don Sanders._______________________________________________KMail developers mailing \
listKMail-devel@kde.orghttps://mail.kde.org/mailman/listinfo/kmail-devel


[prev in list] [next in list] [prev in thread] [next in thread]