[quicwg/base-drafts] Modify QPACK steam and instruction terminology (#2440)

Lucas Pardue <notifications@github.com> Fri, 08 February 2019 17:44 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 23140129741 for <quic-issues@ietfa.amsl.com>; Fri, 8 Feb 2019 09:44:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8
X-Spam-Level:
X-Spam-Status: No, score=-8 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, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] 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 2dAFriEcmNLO for <quic-issues@ietfa.amsl.com>; Fri, 8 Feb 2019 09:44:42 -0800 (PST)
Received: from out-6.smtp.github.com (out-6.smtp.github.com [192.30.252.197]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 21B9B1294FA for <quic-issues@ietf.org>; Fri, 8 Feb 2019 09:44:42 -0800 (PST)
Date: Fri, 08 Feb 2019 09:44:40 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1549647880; bh=1N1vqoWwTNjmu+Wqxo9A0jznt3+NMpJ08SvQmYqkIWo=; h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=p0tBuzSg2/YXoa+dI8K8ueU80HPJeBAEDtHcQXOx6AkI+CXGSqTXzfPeqdqb0Tl8j u9zYTBI5PMHdUd8cz+imo6O05mGuPkgBKamJs1qURCJ09JJbTzaw3Y6nttaVe7CjUE JffHd5n7UiAv6dP+BbDmBcxzWnu8Cpd2nS5kc7vY=
From: Lucas Pardue <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab2285392561e8003de83a4840d871dd8332ee918292cf000000011875820892a169ce18557f5e@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2440@github.com>
Subject: [quicwg/base-drafts] Modify QPACK steam and instruction terminology (#2440)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c5dc008a4c63_66a33fc3586d45c4977e9"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: LPardue
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/UqmkESa-svhkuc8LUAOnvfl9sr0>
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: Fri, 08 Feb 2019 17:44:44 -0000

This PR is more controversial than I first anticipated. It is intended to address #2339.

QPACK uses the term instruction extensively section 4 goes into great detail about all the different types. However, the instructions have been presented in terms of what streams they appear on. Which makes things a little unclear, and verbose, sometimes. I think it also conflates the idea of the true QPACK unframed encoder and decoder streams, with header block messages (sets of instructions).

So this change explicitly creates the terms encoder instruction, decoder instruction and header block instruction. It also removes a lot of the reference to request and push streams.

I'm not 100% happy with the net outcome of these changes but posting the PR for feedback.
You can view, comment on, or merge this pull request online at:

  https://github.com/quicwg/base-drafts/pull/2440

-- Commit Summary --

  * Focus terminology on QPACK instructions rather than stream types
  * Few more tweaks

-- File Changes --

    M draft-ietf-quic-qpack.md (146)

-- Patch Links --

https://github.com/quicwg/base-drafts/pull/2440.patch
https://github.com/quicwg/base-drafts/pull/2440.diff

-- 
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/2440