Re: [quicwg/base-drafts] Core terminology in QPACK (#3520)

afrind <> Tue, 19 May 2020 23:55 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id F3E063A0542 for <>; Tue, 19 May 2020 16:55:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -1.697
X-Spam-Status: No, score=-1.697 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_IMAGE_ONLY_28=1.404, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id TKUIq93TDoHg for <>; Tue, 19 May 2020 16:55:53 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 700583A053F for <>; Tue, 19 May 2020 16:55:53 -0700 (PDT)
Received: from ( []) by (Postfix) with ESMTP id AA55052101D for <>; Tue, 19 May 2020 16:55:52 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1589932552; bh=zA46fM7GxuhImY7YnRDVhDqzkWuK5pxXoayS1Cmevwg=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=pUD9AALXsoLs+0TFR/X26Ccfnrz0VMB/2oN6fLEnzxt6kDNolBY5qYWh5Zn7gDGjW fnFO4k1FVWuUD4l29dqRT3OpR3BQsidcN8u+BmmRECI7pwkELpypHZ/cxxry3HoycV aGASPI/f8lcM27bVifHiqF3lMmNqY7ztABEmmKVo=
Date: Tue, 19 May 2020 16:55:52 -0700
From: afrind <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/3520/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] Core terminology in QPACK (#3520)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5ec472089a391_65d43ff2952cd96423382e"; 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
Archived-At: <>
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Tue, 19 May 2020 23:55:55 -0000

@afrind commented on this pull request.

I think there are few places where we should be using 'encoded field section' rather than just field section.  I made a pass and will update shortly.  It's not perfect, since there are places where it just felt too redundant and field section alone was unambiguous.  I'm open to feedback though.

Also, I changed "Entry Acknowledgement" to "Section Acknowledgement", since that instruction acknowledges processing of an encoded field section, not an individual entry or representation.

> @@ -217,7 +235,7 @@ contains entries which cannot be evicted.
 A dynamic table entry cannot be evicted immediately after insertion, even if it
 has never been referenced. Once the insertion of a dynamic table entry has been
 acknowledged and there are no outstanding references to the entry in
-unacknowledged header blocks, the entry becomes evictable.  Note that
+unacknowledged representations, the entry becomes evictable.  Note that

representations are not acknowledged.   We lost the definition of "header block", and now use 'encoded field section' ?

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: