Robert Wilton's No Objection on draft-ietf-quic-qpack-20: (with COMMENT)

Robert Wilton via Datatracker <noreply@ietf.org> Thu, 21 January 2021 09:38 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: quic@ietf.org
Delivered-To: quic@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id F27753A1860; Thu, 21 Jan 2021 01:38:03 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Robert Wilton via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-quic-qpack@ietf.org, quic-chairs@ietf.org, quic@ietf.org, quic-chairs@ietf.org, lucaspardue.24.7@gmail.com
Subject: Robert Wilton's No Objection on draft-ietf-quic-qpack-20: (with COMMENT)
X-Test-IDTracker: no
X-IETF-IDTracker: 7.24.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Robert Wilton <rwilton@cisco.com>
Message-ID: <161122188396.28810.16954139123191747142@ietfa.amsl.com>
Date: Thu, 21 Jan 2021 01:38:03 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic/iK8oxlS9xkDy2pPmCFjeXzJicTI>
X-BeenThere: quic@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Main mailing list of the IETF QUIC working group <quic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic>, <mailto:quic-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic/>
List-Post: <mailto:quic@ietf.org>
List-Help: <mailto:quic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic>, <mailto:quic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 21 Jan 2021 09:38:04 -0000

Robert Wilton has entered the following ballot position for
draft-ietf-quic-qpack-20: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-quic-qpack/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Thank you for this well written document, like Eric I found it interesting to
read.

A few non blocking comments:

I didn't read the draft in complete detail, but I'm not sure that I came away
with a good understanding of what the receiver data structure would actually
look like in an implementation.  The first paragraph in section 3.2 suggests
that this would be a list of field lines in FIFO order, but I would assume that
a more performant representation would likely be used.  I appreciate that this
is really an implementation detail, but possibly the introduction in section
3.2 might benefit with text giving a bit more overview of what the data
structure is expected to look like.

A couple of places in the document have pseudo code, which I presume is written
in Python.  Possibly, it might be helpful to readers to indicate that the
pseudo code follows a Python style syntax, although it is pretty intuitive
regardless.

Section 7.4 talks about implementation limits, but it wasn't obvious to me how
a receiver is expected to behave if one of those limits is exceeded.  Further
in the case of strings, there seems to be a simple upper bound on the maximum
size of the string of the table size.

I note that the algorithm uses a static table.  Is there any consideration to
be able to update or evolve that static table over time?  E.g., perhaps in 10
years time, the traffic will have changed sufficiently enough that a new
version of the static table should be generated.