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

Mingui Zhang <zhangmingui@huawei.com> Wed, 29 July 2015 01:26 UTC

Return-Path: <zhangmingui@huawei.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 C10171B2E4A for <trill@ietfa.amsl.com>; Tue, 28 Jul 2015 18:26:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.211
X-Spam-Level:
X-Spam-Status: No, score=-4.211 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] 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 A7rfM9xPx2iz for <trill@ietfa.amsl.com>; Tue, 28 Jul 2015 18:26:25 -0700 (PDT)
Received: from dfwrgout.huawei.com (dfwrgout.huawei.com [206.16.17.72]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 309A91A9070 for <trill@ietf.org>; Tue, 28 Jul 2015 18:26:24 -0700 (PDT)
Received: from 172.18.9.243 (EHLO lhreml405-hub.china.huawei.com) ([172.18.9.243]) by dfwrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BGQ37023; Tue, 28 Jul 2015 20:26:23 -0500 (CDT)
Received: from NKGEML404-HUB.china.huawei.com (10.98.56.35) by lhreml405-hub.china.huawei.com (10.201.5.242) with Microsoft SMTP Server (TLS) id 14.3.235.1; Wed, 29 Jul 2015 02:26:19 +0100
Received: from NKGEML512-MBX.china.huawei.com ([169.254.7.227]) by nkgeml404-hub.china.huawei.com ([10.98.56.35]) with mapi id 14.03.0158.001; Wed, 29 Jul 2015 09:26:17 +0800
From: Mingui Zhang <zhangmingui@huawei.com>
To: Michael Richardson <mcr+ietf@sandelman.ca>, "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>, "draft-ietf-trill-aa-multi-attach.all@tools.ietf.org" <draft-ietf-trill-aa-multi-attach.all@tools.ietf.org>, "trill@ietf.org" <trill@ietf.org>
Thread-Topic: RtgDir review: draft-ietf-trill-aa-multi-attach-03
Thread-Index: AQHQyAxZejQVo6xmxEqPqXjEbUB+iJ3wlazQ
Date: Wed, 29 Jul 2015 01:26:16 +0000
Message-ID: <4552F0907735844E9204A62BBDD325E78714C5CA@nkgeml512-mbx.china.huawei.com>
References: <15853.1437871849@dooku.sandelman.ca>
In-Reply-To: <15853.1437871849@dooku.sandelman.ca>
Accept-Language: en-US, zh-CN
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.111.146.93]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <http://mailarchive.ietf.org/arch/msg/trill/dtGigYYDtvHrISz9Fz5tG7UqHuY>
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: Wed, 29 Jul 2015 01:26:32 -0000

Hi Michael,

We really appreciate you review! 

I will reshape the introduction to make it simple. The issues being pointed out are fair. I think I can address them with a new version.

Thanks,
Mingui

> -----Original Message-----
> From: Michael Richardson [mailto:mcr+ietf@sandelman.ca]
> Sent: Sunday, July 26, 2015 8:51 AM
> To: =?utf-8?Q?=E2=80=8Brtg-ads?=@tools.ietf.org;
> draft-ietf-trill-aa-multi-attach.all@tools.ietf.org; trill@ietf.org
> Subject: RtgDir review: draft-ietf-trill-aa-multi-attach-03
> 
> 
> 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>, Sandelman Software Works
> -= IPv6 IoT consulting =-
> 
>