Re: [quicwg/base-drafts] Editorial suggestions on the Streams section (#2730)
Mike Bishop <notifications@github.com> Wed, 19 June 2019 16:29 UTC
Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2D43212004C for <quic-issues@ietfa.amsl.com>; Wed, 19 Jun 2019 09:29:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.009
X-Spam-Level:
X-Spam-Status: No, score=-8.009 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com
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 xqT0ty2_Ulxt for <quic-issues@ietfa.amsl.com>; Wed, 19 Jun 2019 09:29:34 -0700 (PDT)
Received: from out-20.smtp.github.com (out-20.smtp.github.com [192.30.252.203]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A34101201ED for <quic-issues@ietf.org>; Wed, 19 Jun 2019 09:29:33 -0700 (PDT)
Date: Wed, 19 Jun 2019 09:29:32 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1560961772; bh=B+W4vKU5FpIbU64x+gHVd9dVhA8otQVSG5UmPefU8NE=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=f1pLBW2RTw0Y36xX7ebk8jyahsanp2abgdiv6ZKLOySUGSbqzXDJzsDsZnd0L5unz GN6pnkmyj0WGywhqtkZ3/6BIssewxLf7g8tN0bHDkFAkqAeUF+sWxP4aklDPeDTdHl Zs/SQFJSqrsegEy06xesTmN3LPWUlchRTtpenNcw=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK54ASNSKXS7CG73BNF3C6KWZEVBNHHBVGTFHY@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2730/review/251820000@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2730@github.com>
References: <quicwg/base-drafts/pull/2730@github.com>
Subject: Re: [quicwg/base-drafts] Editorial suggestions on the Streams section (#2730)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5d0a62ec83fda_540a3fdc320cd96c364626"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: MikeBishop
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/PgEN5AcgC0MTY4qM7IqIf-9ZQfM>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Jun 2019 16:29:36 -0000
MikeBishop commented on this pull request. > @@ -187,8 +187,9 @@ QUIC: QUIC packet: -: The smallest unit of QUIC that can be encapsulated in a UDP datagram. Multiple - QUIC packets can be encapsulated in a single UDP datagram. +: A complete processable payload with a packet type, encryption level and Structure? Sequence of octets? Or drop it entirely and define a packet as a header, payload, and authentication tag in that order. > @@ -335,8 +337,9 @@ the flow control limits set by its peer. Flow control is described in detail in Stream multiplexing can have a significant effect on application performance if resources allocated to streams are correctly prioritized. -QUIC does not provide frames for exchanging prioritization information. Instead -it relies on receiving priority information from the application that uses QUIC. +QUIC does not provide a mechanism for exchanging prioritization information. +Instead it relies on receiving priority information from the application that ```suggestion Instead, it relies on receiving priority information from the application that ``` > @@ -335,8 +337,9 @@ the flow control limits set by its peer. Flow control is described in detail in Stream multiplexing can have a significant effect on application performance if resources allocated to streams are correctly prioritized. -QUIC does not provide frames for exchanging prioritization information. Instead -it relies on receiving priority information from the application that uses QUIC. +QUIC does not provide a mechanism for exchanging prioritization information. +Instead it relies on receiving priority information from the application that +uses QUIC. implementation-specific, with the hyphen > @@ -543,9 +546,9 @@ MAX_STREAM_DATA frames, it only receives any retransmissions of stream data. Once all data for the stream has been received, the receiving part enters the "Data Recvd" state. This might happen as a result of receiving the same STREAM -frame that causes the transition to "Size Known". In this state, the endpoint -has all stream data. Any STREAM or STREAM_DATA_BLOCKED frames it receives for -the stream can be discarded. +frame that causes the transition to "Size Known". After all data has been +received, Any STREAM or STREAM_DATA_BLOCKED frames it receives for the stream ```suggestion received, any STREAM or STREAM_DATA_BLOCKED frames for the stream ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/quicwg/base-drafts/pull/2730#pullrequestreview-251820000
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- [quicwg/base-drafts] Editorial suggestions on the… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… Martin Thomson
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… MikkelFJ
- Re: [quicwg/base-drafts] Editorial suggestions on… MikkelFJ
- Re: [quicwg/base-drafts] Editorial suggestions on… MikkelFJ
- Re: [quicwg/base-drafts] Editorial suggestions on… Mike Bishop
- Re: [quicwg/base-drafts] Editorial suggestions on… Martin Thomson
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… Mike Bishop
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… Martin Thomson
- Re: [quicwg/base-drafts] Editorial suggestions on… ianswett
- Re: [quicwg/base-drafts] Editorial suggestions on… Martin Thomson