[Gen-art] Gen-ART review: draft-ietf-pwe3-pw-typed-wc-fec-03

Mary Barnes <mary.ietf.barnes@gmail.com> Mon, 09 April 2012 19:21 UTC

Return-Path: <mary.ietf.barnes@gmail.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 95AE421F87B5 for <gen-art@ietfa.amsl.com>; Mon, 9 Apr 2012 12:21:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -103.598
X-Spam-Level:
X-Spam-Status: No, score=-103.598 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-1, 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 C8oRk2mfNq9p for <gen-art@ietfa.amsl.com>; Mon, 9 Apr 2012 12:21:46 -0700 (PDT)
Received: from mail-vx0-f172.google.com (mail-vx0-f172.google.com [209.85.220.172]) by ietfa.amsl.com (Postfix) with ESMTP id 7459221F86D4 for <gen-art@ietf.org>; Mon, 9 Apr 2012 12:21:46 -0700 (PDT)
Received: by vcbfk13 with SMTP id fk13so2420090vcb.31 for <gen-art@ietf.org>; Mon, 09 Apr 2012 12:21:46 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:date:message-id:subject:from:to:cc:content-type; bh=0RRe1100zrBRuPOmHILp1ppC24za7bajngG0u5oYiMk=; b=DRFcTtQK4STJ5n4qKKlo5kz11nCNLyVbchdA8zXxqcAFwg2QGqifcr8vmoIXkKzqZ7 RGF6e1bdbDFgeaNz67B+NjuNyiURLU5Xila9s8SkjCJ+efWzOwdbw9arFmrOHIHCGIoM Gz9Fc8fq3lXvvaloW0FjU370kLFt5BtzRn8eYbZh2v4PAUW0JE2irO2PcsO31hcmOui3 xehyxVYKR8vIaObtlyLzQAcEqPoPV6OnPmLltf0Clz1+u9HcVuy/L2srx2Y/WEZCExHr oGzUv0mBRaLqVJgviOATifuXSoacERI+zU4dGQVUn6NeMjhY2jCwjrusI3nqMWqHHMHS SXoA==
MIME-Version: 1.0
Received: by 10.52.72.169 with SMTP id e9mr3446249vdv.21.1333999305987; Mon, 09 Apr 2012 12:21:45 -0700 (PDT)
Received: by 10.52.68.145 with HTTP; Mon, 9 Apr 2012 12:21:45 -0700 (PDT)
Date: Mon, 09 Apr 2012 14:21:45 -0500
Message-ID: <CAHBDyN4-z57zkkc8_thG+PPPso38vZNZy496xNviS+10BPj6_w@mail.gmail.com>
From: Mary Barnes <mary.ietf.barnes@gmail.com>
To: draft-ietf-pwe3-pw-typed-wc-fec.all@tools.ietf.org
Content-Type: multipart/alternative; boundary="20cf3071d0a2c3978204bd43ea8c"
Cc: gen-art@ietf.org
Subject: [Gen-art] Gen-ART review: draft-ietf-pwe3-pw-typed-wc-fec-03
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: Mon, 09 Apr 2012 19:21:47 -0000

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>.

Please wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-ietf-pwe3-pw-typed-wc-fec-03.txt
Reviewer:  Mary Barnes
Review Date:  9 April 2012
IESG Telechat Date: 12 April 2012

Summary:  Ready with nits.

Comments: there are some editorial nits, many summarized below.  There are
several cases where an article - i.e., "the", "an", etc. is missing.   I
believe these can be fixed in AUTH48 and I imagine the RFC editor will
catch them. But, if the doc needs any other updates, it would be good to
make these fixes at the same time.

Nits:
-----
Section 1:
- first paragraph, second sentence is very awkward and could be rewritten
something like (unless I've misunderstood the objective of that sentence:
OLD:
   This can be used when it is
   desired to request all label bindings for a given type of FEC
   Element, or to release or withdraw all label bindings for a given
   type of FEC element.
NEW:
   This can be used to request, release or withdraw all label
   bindings for a given type of FEC element.

- second paragraph, last sentence - missing articles (PWID, same)
OLD:
   The procedures for Typed Wildcard processing for PWid and
   Generalized PWid FEC Elements are same as described in [RFC5918] for
   any typed wildcard FEC Element type.
NEW:
   The procedures for Typed Wildcard processing for the PWid and
   Generalized PWid FEC Elements are the same as described in [RFC5918] for
   any typed wildcard FEC Element type.

Section 3:
- third paragraph, last sentence before list:
OLD:
 provide
  more generalized and comprehensive solution by allowing:
NEW:
  provide a
  more generalized and comprehensive solution by allowing:

- list item 2: "constraint" -> "constrain"



Section 4.1:
OLD:
 "… had learnt from LSR B over LDP session."
NEW:
 " … learned from the LSR B over the LDP session."

OLD:
 "…such request…"
NEW:
 "…such a request…"  OR "..the request…"

OLD:
 (no stale)
NEW:
 (not stale)

OLD:
  completes consistency check
NEW:
  completes the consistency check

Section 4.3:
- first paragraph, last sentence:
OLD:
  with large number
NEW:
  with a large number

Section 4.4:
- 1st para, 2nd sentence:
OLD:
   These procedures use LDP Address Withdraw message
NEW:
   These procedures use an LDP Address Withdraw message

- 2nd paragraph, last sentence:
OLD:
  This per PW (VPLS instance) MAC Withdraw messages
NEW:
  These per PW (VPLS instance) MAC Withdraw messages

3rd paragraph - 1st sentence:
OLD:
  this document allows use of
NEW:
  this document allows the use of