Re: [sieve] Duplicate test

NED+mta-filters@mauve.mrochek.com Sun, 20 January 2013 22:33 UTC

Return-Path: <NED+mta-filters@mauve.mrochek.com>
X-Original-To: sieve@ietfa.amsl.com
Delivered-To: sieve@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EDF1D21F8801 for <sieve@ietfa.amsl.com>; Sun, 20 Jan 2013 14:33:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0
X-Spam-Level:
X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id niOE4lIN-yQJ for <sieve@ietfa.amsl.com>; Sun, 20 Jan 2013 14:33:36 -0800 (PST)
Received: from mauve.mrochek.com (mauve.mrochek.com [66.59.230.40]) by ietfa.amsl.com (Postfix) with ESMTP id 7B9B621F87FF for <sieve@ietf.org>; Sun, 20 Jan 2013 14:33:33 -0800 (PST)
Received: from dkim-sign.mauve.mrochek.com by mauve.mrochek.com (PMDF V6.1-1 #35243) id <01OP7SPKWJTS005T7L@mauve.mrochek.com> for sieve@ietf.org; Sun, 20 Jan 2013 14:28:31 -0800 (PST)
Received: from mauve.mrochek.com by mauve.mrochek.com (PMDF V6.1-1 #35243) id <01OOJ8U2Q2U800008S@mauve.mrochek.com> (original mail from NED@mauve.mrochek.com) for sieve@ietf.org; Sun, 20 Jan 2013 14:28:26 -0800 (PST)
From: NED+mta-filters@mauve.mrochek.com
Message-id: <01OP7SPIEZWS00008S@mauve.mrochek.com>
Date: Sun, 20 Jan 2013 14:27:53 -0800
In-reply-to: "Your message dated Sun, 20 Jan 2013 14:00:36 +0100" <50FBEA74.90603@rename-it.nl>
MIME-version: 1.0
Content-type: TEXT/PLAIN; format="flowed"
References: <50F3F8C7.9030804@rename-it.nl> <50F72B63.3080004@isode.com> <01OP4U4XTT0U00008S@mauve.mrochek.com> <50FBEA74.90603@rename-it.nl>
To: Stephan Bosch <stephan@rename-it.nl>
Cc: Ned Freed <ned.freed@mrochek.com>, Sieve mailing list <sieve@ietf.org>
Subject: Re: [sieve] Duplicate test
X-BeenThere: sieve@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: SIEVE Working Group <sieve.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sieve>, <mailto:sieve-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sieve>
List-Post: <mailto:sieve@ietf.org>
List-Help: <mailto:sieve-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sieve>, <mailto:sieve-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 20 Jan 2013 22:33:37 -0000

> On 1/18/2013 8:22 PM, Ned Freed wrote:
> >> 2). I found that "the side effect of the test takes force at the end of
> >> a successful script execution" to be odd/awkward. Maybe we can have an
> >> explicit action? Or maybe leaving this as is is Ok... Need to think a
> >> bit more about that.
> >
> > I'm not wild about this either, mostly because when you do this as two
> > steps
> > there's a window where overlapping script executions can cause a
> > duplicate to
> > be missed. Of course locking can be used to prevent this, but given
> > the scale
> > we operate at any use of locking has to be carefully considered.
> >
> > That said, the problem where a script goes wonky and ends up marking
> > something
> > as a duplicate is much more severe, so IMO the draft is handling this
> > the right
> > way.
> >
> > tl;dr: Missing a duplicate is preferable to marking a message as a
> > duplicate
> > incorrectly.

> Exactly.

> Perhaps I should mention these considerations more explicitly in the draft?

Yes, I think that's a good idea.

				Ned