Re: [Gen-art] [Taps] Genart last call review of draft-ietf-taps-minset-06

Robert Sparks <rjsparks@nostrum.com> Tue, 04 September 2018 17:57 UTC

Return-Path: <rjsparks@nostrum.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 75ACB12F1A5; Tue, 4 Sep 2018 10:57:52 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.878
X-Spam-Level:
X-Spam-Status: No, score=-1.878 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 0K7lWFVQzUl5; Tue, 4 Sep 2018 10:57:48 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (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 85D9D130EDC; Tue, 4 Sep 2018 10:57:48 -0700 (PDT)
Received: from unescapeable.local ([47.186.18.66]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id w84Hvi9S098429 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Tue, 4 Sep 2018 12:57:44 -0500 (CDT) (envelope-from rjsparks@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host [47.186.18.66] claimed to be unescapeable.local
To: Spencer Dawkins at IETF <spencerdawkins.ietf@gmail.com>, Michael Welzl <michawe@ifi.uio.no>
Cc: gen-art <gen-art@ietf.org>, draft-ietf-taps-minset.all@ietf.org, IETF list <ietf@ietf.org>, "taps@ietf.org" <taps@ietf.org>
References: <153549231960.6181.5129820855641929896@ietfa.amsl.com> <526D1CA4-DD3D-4AEE-8B92-B120A6798C53@ifi.uio.no> <CAKKJt-f3ZBL83mHiWjknyT4s+Vfa5HoCS+wkA-=v3JAWK1didw@mail.gmail.com> <A2E874E0-4638-464F-A027-DA7C8BEE5BAE@ifi.uio.no> <CAKKJt-coUjEL++_Cq37aTuCWFjSVXpftHDpCBrGh6GdUevDnyA@mail.gmail.com>
From: Robert Sparks <rjsparks@nostrum.com>
Message-ID: <8e74a20d-c1c3-19fa-aa0d-76a5a090902f@nostrum.com>
Date: Tue, 04 Sep 2018 12:57:43 -0500
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <CAKKJt-coUjEL++_Cq37aTuCWFjSVXpftHDpCBrGh6GdUevDnyA@mail.gmail.com>
Content-Type: multipart/alternative; boundary="------------FB546FAD6BE3BDE406845698"
Content-Language: en-US
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/fAV1eUERvlXBStX_ydhP9jCrkSM>
Subject: Re: [Gen-art] [Taps] Genart last call review of draft-ietf-taps-minset-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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, 04 Sep 2018 17:57:53 -0000


On 9/1/18 10:43 PM, Spencer Dawkins at IETF wrote:
> Hi, Michael,
>
> On Fri, Aug 31, 2018, 15:35 Michael Welzl <michawe@ifi.uio.no 
> <mailto:michawe@ifi.uio.no>> wrote:
>
>     Hi Spencer,
>
>     See below:
>
>>     On Aug 31, 2018, at 7:41 PM, Spencer Dawkins at IETF
>>     <spencerdawkins.ietf@gmail.com
>>     <mailto:spencerdawkins.ietf@gmail.com>> wrote:
>>
>>     Thanks, Robert, for the careful read, and thanks, Michael, for
>>     the quick response. I have one thought, on Robert's last question.
>>
>>     On Fri, Aug 31, 2018 at 3:37 AM Michael Welzl <michawe@ifi.uio.no
>>     <mailto:michawe@ifi.uio.no>> wrote:
>>
>>         Hi,
>>
>>         Thank you very much for this careful review!  We just posted
>>         a revision ( -07 ) that, we believe, addresses these comments.
>>
>>         A few answers in line below:
>>
>>         > On 28 Aug 2018, at 23:38, Robert Sparks
>>         <rjsparks@nostrum.com <mailto:rjsparks@nostrum.com>> wrote:
>>         >
>>         > Reviewer: Robert Sparks
>>         > Review result: Ready with Nits
>>
>>
>>     ...
>>
>>         > In Appendix A.1, this document had to "slightly change" the
>>         > characterization of features  from those in RFC8303,
>>         introducing this
>>         > "ADDED" construct. That feels out of balance. Is this a
>>         warning sign
>>         > that RFC8303 needs adjusting?
>>
>>         It isn't: different from this document, RFC 8303 does not
>>         make any changes or derive anything from the services that
>>         protocols offer - it just reflects what the protocol
>>         specifications say.
>>
>>         In the minset document, there are only 3 items using the
>>         "ADDED" construct, and this is only meant to streamline the
>>         derivation a little. Take "Unreliably transfer a message",
>>         for example.
>>         This is based on (from RFC 8303) "Unreliably transfer a
>>         message, with congestion control" for SCTP, and "Unreliably
>>         transfer a message, without congestion control" for
>>         UDP(-Lite). The added "Unreliably transfer a message" leaves
>>         the choice of congestion control open, such that an
>>         application CAN simply say "Unreliably transfer a message"
>>         without having to care about the choice of congestion control
>>         (unless it wants to care, which comes at the cost of binding
>>         itself to either UDP(-Lite) or SCTP).
>>
>>
>>     Michael, this explanation helps a lot, but since I happen to know
>>     that Robert is out of town for the three-day weekend in the US,
>>     I'll guess on his behalf that "ADDED" may not be the word you're
>>     looking for - at a minimum, "transfer unreliably" in RFC 8303
>>     being divided into "transfer unreliably with congestion control"
>>     and "transfer unreliably without congestion control" in this
>>     draft doesn't look like addition to me.
>>
>>     Is this more about "refactoring the protocol-independent
>>     definition in RFC 8303”?
>
>     Yes, “refactoring” is exactly right - it’s not adding anything
>     new. We could just as well have left this without the “ADDED”
>     cases and then had more explanations in the “discussions” section
>     (appendix A.3), but these are so minor that they don’t really
>     merit a “discussion”, hence we felt that this way it’s shorter and
>     clearer.
>
>     If that helps, I can rename “ADDED” into “REFACTORED”?
>
With that explanation, the single word without the explanation above 
feels like insider knowledge.
Maybe adding a sentence explaining exactly what you say above at the 
point you introduce the term would be enough, then the term-name itself 
wouldn't matter as much.

>
> I'll let you and Robert resynchronize on this, now that I think you'll 
> be talking about the same thing.
>
> I'll watch, if I'm needed, of course.
>
> Spencer
>
>     Cheers,
>     Michael
>>     But, whatever it is, if you two can figure out how to describe
>>     what's happening, that will probably help figure you and Robert
>>     agree on an understanding about how to handle his comment.
>>
>>     Spencer
>
>