Re: [quicwg/base-drafts] Acknowledge stream resets too (#1400)

afrind <notifications@github.com> Fri, 01 June 2018 21:43 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 A53CE12DA54 for <quic-issues@ietfa.amsl.com>; Fri, 1 Jun 2018 14:43:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.01
X-Spam-Level:
X-Spam-Status: No, score=-8.01 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_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 ATnwSZr4LIKQ for <quic-issues@ietfa.amsl.com>; Fri, 1 Jun 2018 14:43:32 -0700 (PDT)
Received: from out-3.smtp.github.com (out-3.smtp.github.com [192.30.252.194]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id CA22112DA1C for <quic-issues@ietf.org>; Fri, 1 Jun 2018 14:43:32 -0700 (PDT)
Date: Fri, 01 Jun 2018 14:43:31 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1527889411; bh=bEag79AYjY5CpKPTDHyCGC0SlU2VwxpJLQykK+c0wPw=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=BER9t8+vfcVF3Yo4eoHfBUmygkLNHzAxtvDbDJjRQntF0TgonvuvsYrwS/KMbQQla Hk49TVBD6NMoCvRcXI9WP/VB+loCRmzGzOnEdQAItMwy0bc9oOKPOWIY7yFvPl5ooJ nUY8ChrO7rHIDyNMXzRa2n07xsKyinI32mgzyqdE=
From: afrind <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab2ad2666687575f6190722a588ce2b20fbfc43d7692cf000000011729800392a169ce138608d0@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1400/review/125325455@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1400@github.com>
References: <quicwg/base-drafts/pull/1400@github.com>
Subject: Re: [quicwg/base-drafts] Acknowledge stream resets too (#1400)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b11be03edd32_6d223fc45b4eef80280a4"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: afrind
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/hsqJIKiB5hsw31J2UDTV9eDBI30>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
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: Fri, 01 Jun 2018 21:43:35 -0000

afrind commented on this pull request.



> @@ -536,6 +536,31 @@ instruction is also processed as a Table Size Synchronize instruction with a
 value matching the Largest Reference of the corresponding header block.
 
 
+### Stream Cancellation
+
+A stream that is reset might have multiple outstanding header blocks.  A decoder
+that receives a stream reset before the end of a stream generates a Stream
+Cancellation instruction on the decoder stream.  Similarly, a decoder that
+abandons reading of a stream needs to signal this using the Stream Cancellation
+instruction.  This signals to the encoder that all references to the dynamic
+table on that stream are no longer outstanding.
+
+An encoder cannot infer from this acknowledgement that any dynamic table entries
+referenced have been received.

I think you're trying to say that the encoder can't make any assumptions about the state of the dynamic table, as can be done with header acknowledgement.

"...cannot infer from this acknowledgement any updates to the dynamic table have been received" ?

-- 
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/1400#pullrequestreview-125325455