Murray Kucherawy's Yes on draft-ietf-quic-qpack-20: (with COMMENT)

Murray Kucherawy via Datatracker <noreply@ietf.org> Thu, 21 January 2021 06:33 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 3C3F53A0E5C; Wed, 20 Jan 2021 22:33:03 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Murray Kucherawy 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: Murray Kucherawy's Yes 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: Murray Kucherawy <superuser@gmail.com>
Message-ID: <161121078322.7962.13407479542483381007@ietfa.amsl.com>
Date: Wed, 20 Jan 2021 22:33:03 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic/UxQhCu4pFtLi7v5IVmTVjCQlHuU>
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 06:33:03 -0000

Murray Kucherawy has entered the following ballot position for
draft-ietf-quic-qpack-20: Yes

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:
----------------------------------------------------------------------

Kudos on all the hard work that clearly went into this.  Makes our job easy,
and this was also very educational.

Some minor suggestions:

In Section 2.1.3, I suggest "SHOULD avoid writing" ought to be "SHOULD NOT
write". (For that matter, I don't know why this isn't a MUST NOT.)

In Section 2.2.1:

   While blocked, encoded field section data SHOULD remain in the
   blocked stream's flow control window.

Since SHOULD grants me a choice, why might I decide not to do what it says
here?  Or should this be MUST?