Re: [trill] RtgDir review: draft-ietf-trill-aa-multi-attach-03

Alia Atlas <akatlas@gmail.com> Mon, 27 July 2015 18:19 UTC

Return-Path: <akatlas@gmail.com>
X-Original-To: trill@ietfa.amsl.com
Delivered-To: trill@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3A15E1AC44A for <trill@ietfa.amsl.com>; Mon, 27 Jul 2015 11:19:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.999
X-Spam-Level:
X-Spam-Status: No, score=-101.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0n2ZR405MOl5 for <trill@ietfa.amsl.com>; Mon, 27 Jul 2015 11:19:32 -0700 (PDT)
Received: from mail-oi0-x236.google.com (mail-oi0-x236.google.com [IPv6:2607:f8b0:4003:c06::236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id EA4251B3187 for <trill@ietf.org>; Mon, 27 Jul 2015 11:19:31 -0700 (PDT)
Received: by oibn4 with SMTP id n4so56039561oib.3 for <trill@ietf.org>; Mon, 27 Jul 2015 11:19:31 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=hq8Ap/d0mp1kE7G/VpX9LDOox7QD+CKCPZEub6tD1v0=; b=02N8SFWxLMNccx3qZBypJQPRRJ2Au0hrz5JuO8w8W2/Z6JhubV6RAILbdSOigQsO7P Hi533qIOOU7qPdhijAJ6Zr6PX6FD/mNAOY+TPwvHtopItI/QsOu3N5u27SzvoHu60iCy HFS8b9Ar3JSFGMTJf5jyjiefk/xtnPXDzFxN5ZMvE9sOw0oqUDOgNIJ8FHG5K+d67ScC oCux9ypzDim2VaMw5afvyWRFABLutBBrGB4C8iFTL0qZkpsRS/wpDiGPctfg9Audark5 8JDbOI4citk4XNmAvkgebD0CDvD/d/ZXUd4WU11x/kM8RhF7t/fXmznMqn51GZsYWifN KgJA==
MIME-Version: 1.0
X-Received: by 10.202.171.21 with SMTP id u21mr28401961oie.113.1438021171386; Mon, 27 Jul 2015 11:19:31 -0700 (PDT)
Received: by 10.60.41.99 with HTTP; Mon, 27 Jul 2015 11:19:31 -0700 (PDT)
In-Reply-To: <15853.1437871849@dooku.sandelman.ca>
References: <15853.1437871849@dooku.sandelman.ca>
Date: Mon, 27 Jul 2015 14:19:31 -0400
Message-ID: <CAG4d1rey9QoDAghCGJrFKVFAhw55QdKiuhymDnV67SyJbubQnQ@mail.gmail.com>
From: Alia Atlas <akatlas@gmail.com>
To: Michael Richardson <mcr+ietf@sandelman.ca>
Content-Type: multipart/alternative; boundary=001a113cae6819a000051bdf6525
Archived-At: <http://mailarchive.ietf.org/arch/msg/trill/9QMAY54iiheEzFOKPLdv59QdA7M>
Cc: draft-ietf-trill-aa-multi-attach.all@tools.ietf.org, trill@ietf.org, "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>
Subject: Re: [trill] RtgDir review: draft-ietf-trill-aa-multi-attach-03
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Developing a hybrid router/bridge." <trill.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/trill>, <mailto:trill-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/trill/>
List-Post: <mailto:trill@ietf.org>
List-Help: <mailto:trill-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/trill>, <mailto:trill-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 27 Jul 2015 18:19:34 -0000

Michael,

Thanks very much for the review.

Regards,
Alia

On Sat, Jul 25, 2015 at 8:50 PM, Michael Richardson <mcr+ietf@sandelman.ca>
wrote:

>
> Hello,
>
> I have been selected as the Routing Directorate reviewer for this draft.
> The
> Routing Directorate seeks to review all routing or routing-related drafts
> as
> they pass through IETF last call and IESG review, and sometimes on special
> request. The purpose of the review is to provide assistance to the Routing
> ADs. For more information about the Routing Directorate, please see
> ​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir
>
> Although these comments are primarily for the use of the Routing ADs, it
> would be helpful if you could consider them along with any other IETF Last
> Call comments that you receive, and strive to resolve them through
> discussion
> or by updating the draft.
>
> Document: draft-ietf-trill-aa-multi-attach-03.txt
> Reviewer: Michael Richardson
> Review Date: 2015-07-25
> IETF LC End Date: N/A
> Intended Status: Proposed Standard
>
> Summary:
>
> This document is basically ready for publication, but could benefit from
> some simplication of language. The forest is often lost for the inline
> details of the trees.  The world would not end if it was published as is.
>
>
> Comments:
>
> Given that the introduction precedes the glossary, it is very TLA heavy.
> I wonder if the Introduction could be simplified, or if it is even
> necessary to say so much so precisely.
>
> Terms that stood out without definition: clumps.
> (I suspect that this is a TRILL core term)
>
> Minor Issues:
>
> I find that the english is rough in places, perhaps a some words or
> half sentences obvious to the authors is missing. e.g. last paragraph
> before section 4.  Sentences seem needlessly long.
>
> If I were implementing in an Agile setting, I'd be writing multiple user
> stories per sentence.  Consider the development manager trying to reconcile
> tickets back to parts of the document.
>
> I'd rather that more periods and more new paragraphs were used to explain
> things.  I don't consider this an english issue, rather an issue of
> too many nested ideas:
>          (A (and also B (except C (unless D))) then E)
>
> I think the document was started with the idea of trying to explain why
> Option C has been chosen by walking the reader through all the choices, and
> comparing them.  Yet, that actually isn't what is happening: the options
> are
> not well enough explained to understand the path not taken.  Either say
> less
> about options A and B, or say more.
>
> I found section 5 (meeting design goals), while dense, to be very well
> written and presented. Kudos.
>
> No major issues found.
>
>
>
> --
> Michael Richardson <mcr+IETF@sandelman.ca>ca>, Sandelman Software Works
>  -= IPv6 IoT consulting =-
>
>
>
>