Re: [Gen-art] Gen-Art review of draft-ietf-appsawg-greylisting

"Murray S. Kucherawy" <msk@cloudmark.com> Tue, 24 April 2012 22:28 UTC

Return-Path: <msk@cloudmark.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2714521F86D8 for <gen-art@ietfa.amsl.com>; Tue, 24 Apr 2012 15:28:11 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.552
X-Spam-Level:
X-Spam-Status: No, score=-102.552 tagged_above=-999 required=5 tests=[AWL=0.047, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id cDzaeF5q-+io for <gen-art@ietfa.amsl.com>; Tue, 24 Apr 2012 15:28:10 -0700 (PDT)
Received: from mail.cloudmark.com (cmgw1.cloudmark.com [208.83.136.25]) by ietfa.amsl.com (Postfix) with ESMTP id 4E21121F86D6 for <gen-art@ietf.org>; Tue, 24 Apr 2012 15:28:09 -0700 (PDT)
Received: from ht1-outbound.cloudmark.com ([72.5.239.25]) by mail.cloudmark.com with bizsmtp id 1yUL1j0010ZaKgw01yULRw; Tue, 24 Apr 2012 15:28:29 -0700
X-CMAE-Match: 0
X-CMAE-Score: 0.00
X-CMAE-Analysis: v=2.0 cv=fNu7LOme c=1 sm=1 a=LdFkGDrDWH2mcjCZERnC4w==:17 a=LvckAehuu68A:10 a=XDoa0A5SzUQA:10 a=zutiEJmiVI4A:10 a=kj9zAlcOel0A:10 a=xqWC_Br6kY4A:10 a=G0_B3m8xAAAA:8 a=48vgC7mUAAAA:8 a=k7Ga1wGzAAAA:8 a=eWNgu4SYTvBnYhoLPl0A:9 a=bNehmqa9pE8moRC2vJUA:7 a=CjuIK1q_8ugA:10 a=lW_bInUQU2sA:10 a=lZB815dzVvQA:10 a=fcAx7uNQz4EA:10 a=LdFkGDrDWH2mcjCZERnC4w==:117
Received: from EXCH-MBX901.corp.cloudmark.com ([fe80::addf:849a:f71c:4a82]) by exch-htcas901.corp.cloudmark.com ([fe80::2524:76b6:a865:539c%10]) with mapi id 14.01.0355.002; Tue, 24 Apr 2012 15:27:59 -0700
From: "Murray S. Kucherawy" <msk@cloudmark.com>
To: "kathleen.moriarty@emc.com" <kathleen.moriarty@emc.com>, "draft-ietf-appsawg-greylisting.all@tools.ietf.org" <draft-ietf-appsawg-greylisting.all@tools.ietf.org>, "iesg@ietf.org" <iesg@ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-Art review of draft-ietf-appsawg-greylisting
Thread-Index: Ac0hcmcyQR3NBVo/Rl6IFM4VkVIGNwA9oqgQ
Date: Tue, 24 Apr 2012 22:27:58 +0000
Message-ID: <9452079D1A51524AA5749AD23E0039281017B8@exch-mbx901.corp.cloudmark.com>
References: <AE31510960917D478171C79369B660FA0E7CB761E7@MX06A.corp.emc.com>
In-Reply-To: <AE31510960917D478171C79369B660FA0E7CB761E7@MX06A.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [172.20.2.121]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cloudmark.com; s=default; t=1335306509; bh=h2zl95gZNOhfY73Vm9wK0GuCNx5dYc+GcDU1ngI6saI=; h=From:To:CC:Subject:Date:Message-ID:References:In-Reply-To: Content-Type:Content-Transfer-Encoding:MIME-Version; b=HbmvBfv6RBFaMDiJcCm83m4vigWDgout9kwSP+IsHYHxQua+p3oaFMCg0ZC6aUD2c h3+CNXgnkznqmecKRhxDN9eHyGyij8K5Qenwx+RtQvEJ2qb11rD08VBT5cJYclyEbe fXqwRYVDo6lMDiB3EomrrpUbQH+vnV/AECc5vCpo=
Cc: "dcrocker@bbiw.net" <dcrocker@bbiw.net>
Subject: Re: [Gen-art] Gen-Art review of draft-ietf-appsawg-greylisting
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Apr 2012 22:28:11 -0000

Hi Kathleen, thanks for the review.

> -----Original Message-----
> From: kathleen.moriarty@emc.com [mailto:kathleen.moriarty@emc.com]
> Sent: Monday, April 23, 2012 9:59 AM
> To: draft-ietf-appsawg-greylisting.all@tools.ietf.org; iesg@ietf.org; gen-art@ietf.org
> Cc: Murray S. Kucherawy; dcrocker@bbiw.net
> Subject: Gen-Art review of draft-ietf-appsawg-greylisting
> 
> I am the assigned Gen-ART reviewer for this draft. For background on
> Gen-ART, please see the FAQ at <
> http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>gt;.
> 
> Please wait for direction from your document shepherd or AD before
> posting a new version of the draft.
> [...]
> Nits/editorial comments:
> Introduction: First sentence:  The word 'differential' should be
> 'different' unless you change the sentence structure.  It could be that
> there is a differential between services.

"different qualities of service" is more like what we want here, because it's the same service really.

> Second sentence:  Change provider to providing

Fixed.

> Section 2.1: The double dashes should be parentheses instead -- --
> replaced by ()

Done.

> Section 3, third paragraph, last sentence: add a comma:
> From: To the greylisting
>    implementation such clients are once again unknown, and they will
>    once again be subjected to the delay.
> To: To the greylisting
>    Implementation, such clients are once again unknown, and they will
>    once again be subjected to the delay.

Done.

> Section 3, fourth paragraph, second sentence: remove the comma:
> From: It has to be
>    large enough to keep the necessary history, and fast enough to avoid
>    excessive inefficiencies in the server's operations.
> To: It has to be
>    large enough to keep the necessary history and fast enough to avoid
>    excessive inefficiencies in the server's operations.

Done.

> Section 3, fifth paragraph, second to last sentence: remove "the," and
> "such"
> From: Experience suggests that
>    the, the vast majority of mail comes from places on such a developed
>    exception list, so after a training period, only a small proportion
>    of mail is actually affected.
> To: Experience suggests that
>    the vast majority of mail comes from places on a developed
>    exception list, so after a training period, only a small proportion
>    of mail is actually affected.

Done.

> Section 4.1: second to last paragraph: remove unnecessary comma:
> From: Other side effects include out-of-order delivery of related,
>    sequenced messages.
> To: Other side effects include out-of-order delivery of related
>    sequenced messages.

OK.

> Section 4.2: Second paragraph:  The commas can be removed as only two
> items are joined in the list.  When you have three or more, a comma is
> used.

Done.

-MSK