Re: Unidirectional streams PR

Ian Swett <ianswett@google.com> Thu, 29 June 2017 01:23 UTC

Return-Path: <ianswett@google.com>
X-Original-To: quic@ietfa.amsl.com
Delivered-To: quic@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 533F8127337 for <quic@ietfa.amsl.com>; Wed, 28 Jun 2017 18:23:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 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, RCVD_IN_DNSWL_LOW=-0.7, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=google.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 uYVGIsCtcNGb for <quic@ietfa.amsl.com>; Wed, 28 Jun 2017 18:23:35 -0700 (PDT)
Received: from mail-yw0-x236.google.com (mail-yw0-x236.google.com [IPv6:2607:f8b0:4002:c05::236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DE74B126E64 for <quic@ietf.org>; Wed, 28 Jun 2017 18:23:34 -0700 (PDT)
Received: by mail-yw0-x236.google.com with SMTP id 63so31455236ywr.0 for <quic@ietf.org>; Wed, 28 Jun 2017 18:23:34 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=zaZ2qLGronWFkrtSGUtnA4NBzMi4GoACB4KveAw6+VU=; b=LCN5RhYIT6WLXvtwzLfeDSkTqh/7czOsVau5Otw6DUFa/co01bY9kFVl4fQ8oZJy/D bKUsGUGklpuDrPV4dHq60/QmDeli0yUjvSoZKo6lkasKd/QPdu84Z3NJu83qG3QjP5Op SkrnX2Mj17toTLW0iXyl9O+w27pJxGhxc/gHOXHOqpJAREPxCcSh9GqohF2ud30dhmAS cIAbbT8x0av1U2zYDHOBgwtIVJ73PrmizM2/kP7ws6qowO1Gt6j+4V+Az0MyqZvYq0W9 UR1Tz5qQi7Wt/4FYKE26ua65qV5iT68gXvyGq0iyYD7NB0nShZqAkBSj95ABEEIkvpL6 0uyg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=zaZ2qLGronWFkrtSGUtnA4NBzMi4GoACB4KveAw6+VU=; b=rjKhMEHJU9YrpfrHOucZMLRORiOcCsLQSyVaZ3irIm3AhKWDq/PVo4BEJrEy2cDdRl JJ9CiEhWsjuuyqNUN2mRhqK2fchBBn3frKCgc8UzNO63/YbHbBEfkMGxXgveLdBlyn71 Q/R1OQf6jDvxNPV/i/JAjfq3J8UglC+BL+VuaUF6NmNnvJjO56P+sVPec+9Oy/bkkpTq gGyT/y0ahCBBtwrbP+ul4LCOt3M6L+4/mCn6qM6I5nOscnxgUXoV88hB5+3y2LHHw9DJ wQOli79egVRheHTWF6QNNtUQFGSluCfu7dxCjGborWWZskH+FoTNFkLvzAn0eo0NGtVI C1Jg==
X-Gm-Message-State: AKS2vOzE8FT22NgD2k10V6mmJTkRoU3qMnvJ06OdZ0aUVo2+/oCD9YyP ovMwybisCB1U4eavCD6UyVM7Qz1pbdbx
X-Received: by 10.129.146.15 with SMTP id j15mr9833836ywg.283.1498699413894; Wed, 28 Jun 2017 18:23:33 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.37.208.3 with HTTP; Wed, 28 Jun 2017 18:23:13 -0700 (PDT)
In-Reply-To: <2240c2a68910453e97fc50d42e8a1d4f@usma1ex-dag1mb5.msg.corp.akamai.com>
References: <CAN1APdc_ckZu39ZZTETv04iZieogoE_NQCBR-n0jHrC-9dM7Aw@mail.gmail.com> <5d69489d-8f46-ebbe-4e5c-fa6c02ffd8dd@huitema.net> <CAF4GZgBm7525i2GxiN-Pv66g0WqbDH==fRXN27=7ursNA70w1Q@mail.gmail.com> <20170628124221.GA15608@ubuntu-dmitri> <CAN1APdc3YO4-FEc6C--PzFGxzQiAUeBZ96HkjtjS1RR0qigrzw@mail.gmail.com> <CAE=ybzNtSZx9-bj9-n-ieLMB=YvJCjCExugvA3_JPVrdEEqK9A@mail.gmail.com> <DB5PR07MB123748F2AB7374DAC0CC9E1484DD0@DB5PR07MB1237.eurprd07.prod.outlook.com> <MWHPR21MB0141BD23011EB26F882C864787DD0@MWHPR21MB0141.namprd21.prod.outlook.com> <CABkgnnXEq9-jxedU_Rmi4XQ+t0SNUOAMbyWXcnhyLKz+OzP2CQ@mail.gmail.com> <2240c2a68910453e97fc50d42e8a1d4f@usma1ex-dag1mb5.msg.corp.akamai.com>
From: Ian Swett <ianswett@google.com>
Date: Wed, 28 Jun 2017 21:23:13 -0400
Message-ID: <CAKcm_gMb9PkBKhTRF3ue2KGgwHgKN8rsanD8rqqr_wUFJ3GNZQ@mail.gmail.com>
Subject: Re: Unidirectional streams PR
To: "Lubashev, Igor" <ilubashe@akamai.com>
Cc: Martin Thomson <martin.thomson@gmail.com>, Mike Bishop <Michael.Bishop@microsoft.com>, "Swindells, Thomas (Nokia - GB/Cambridge, UK)" <thomas.swindells@nokia.com>, QUIC WG <quic@ietf.org>, Jo Kulik <jokulik@google.com>, Mikkel Fahnøe Jørgensen <mikkelfj@gmail.com>, Dmitri Tikhonov <dtikhonov@litespeedtech.com>
Content-Type: multipart/alternative; boundary="94eb2c094216311c7005530f268f"
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic/LUCkvMlcro6Vxbj2JCi7pdLEXu8>
X-BeenThere: quic@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
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, 29 Jun 2017 01:23:39 -0000

I updated my PR(#656 <https://github.com/quicwg/base-drafts/pull/656>)
today, sorry for the delay.  I attempted to address the issues identified
with text.  Some may prefer more tweaks to the state diagram, which are
also possible, so I'm open to suggestions.

In the meantime, I've been considering other alternatives, including
variations on Mike's direction and a variation of the "Do Nothing" option
which involved the application signaling to the transport that streams of a
certain sort(ie: server to client for server push) were unidirectional, as
GQUIC does today.  Overall, I think the approach I've outlined does a good
job of iterating on existing deployment experience and adding explicit
signaling for unidirectional streams instead of implicit signaling at the
application layer.  There are pros and cons to both explicit and implicit
signaling, but I think explicit signaling is more complete and less prone
to application error.

Thanks, Ian


On Wed, Jun 28, 2017 at 8:14 PM, Lubashev, Igor <ilubashe@akamai.com> wrote:

> > unless what the transport provides is a perfect fit for application
> semantics, you end up building those semantics into the application anyway.
>
> I agree with this. We should avoid adding complexity into transport for
> rare use cases, since it goes against KISS principle.
>
> On the other hand, adding support for a by far the most common use case
> makes a lot of sense.  This helps apps avoid screwing up implementing that
> common case and lets us optimize that common case in the lower layer. BiDi
> streams are such common cases. Uni streams are likely to be the
> second-most-common cases (hence you offered this PR to optimize them).
>
>
> The Associated Streams proposal offers extra semantic flexibility at a
> cost of some semantic complexity (someone would need to verify that the
> associated stream numbers make sense -- api? apps?) and a few extra bytes.
>
> I'd like to wait to see Ian's revised proposal.  The initial proposal
> offered to do only one thing -- offer a choice of uni/bi-directional
> streams -- but it did it in a very simple way, which is nice.
>
> - Igor
>
>
> -----Original Message-----
> From: Martin Thomson [mailto:martin.thomson@gmail.com]
> Sent: Wednesday, June 28, 2017 7:28 PM
> To: Mike Bishop <Michael.Bishop@microsoft.com>
> Cc: Swindells, Thomas (Nokia - GB/Cambridge, UK) <
> thomas.swindells@nokia.com>; QUIC WG <quic@ietf.org>; Mikkel Fahnøe
> Jørgensen <mikkelfj@gmail.com>; Dmitri Tikhonov <
> dtikhonov@litespeedtech.com>; Jo Kulik <jokulik@google.com>
> Subject: Re: Unidirectional streams PR
>
> There is probably a simpler approach here, take a bit (as Ian did) and say
> that if that bit is set, then the stream is in response to another and the
> stream ID of the stream to which this is responding follows immediately
> after the stream ID of the stream itself.  You could then include that only
> at the start of the stream, or in multiple frames (or as we decide).
>
> The problem with this, as with several of the other issues we're
> discussing, is that unless what the transport provides is a perfect fit for
> application semantics, you end up building those semantics into the
> application anyway.  HTTP certainly can't survive without its own
> association semantics for pushes.  That suggests to me that having
> bidirectional semantics in the transport creates more duplication than
> otherwise.  Hence my proposal.
>
> On 28 June 2017 at 15:26, Mike Bishop <Michael.Bishop@microsoft.com>
> wrote:
> > As promised, a PR for adding “associated streams” is at
> > https://github.com/quicwg/base-drafts/pull/672.  This very
> > deliberately builds on top of MT’s PR – it’s adding a primitive which
> > can be used to construct various abstractions atop unidirectional
> > streams, but the lifecycle is still fundamentally unidirectional.
> >
> >
> >
> > Copying my notes here for list discussion purposes.
> >
> > Major changes
> >
> > Leveraging @igorlord's insight that OO=00 only occurs on the first
> > STREAM frame of a stream, I used that as the trigger for a Stream
> Properties byte.
> > Two bits of that byte describe the directionality of the stream:
> >
> > Unidirectional (no response expected)
> > Initial bidirectional (one response expected) Initial multi-response
> > (one or more responses expected; needs a better name) Response
> >
> > If the type is Response, there's an Associated Stream ID field, length
> > given by two more bits following the same pattern as the SS bits in
> > the STREAM frame ID.
> >
> > Personal Opinion
> >
> > On the plus side, these stream types seem to cover the abstractions I
> > can envision for most applications. You can unilaterally send
> > something (unidirectional), do request/response (bidirectional), or
> > pub/sub (single subscription stream, series of update streams).
> >
> > I don't care for the fact that I still need the stream type header in
> > HTTP after putting this in the transport. That will be ameliorated if
> > we go back to one stream per request, since all unidirectional streams
> > will be push streams. (As a side-note, I considered using the
> > multiple-response option in the HTTP mapping, but then I need a stream
> > header again to indicate which is the response and which the pushes.)
> >
> > I particularly don't like that you now have to look at the frame type
> > header to find out whether a field exists which tells you the length
> > of something else in the header. I'd like to simplify that. I went
> > with this model over a CREATE_STREAM frame because of @mikkelfj's
> > use-case of very small messages
> > -- this adds only one byte to the first frame on a stream in one
> > direction and 2-5 bytes to the first frame of response streams. A
> > separate frame type would be somewhat larger, but could be cleaner in
> that respect.
> >
> >
> >
> >
> >
> > From: QUIC [mailto:quic-bounces@ietf.org] On Behalf Of Swindells,
> > Thomas (Nokia - GB/Cambridge, UK)
> > Sent: Wednesday, June 28, 2017 7:56 AM
> > To: Jo Kulik <jokulik@google.com>; Mikkel Fahnøe Jørgensen
> > <mikkelfj@gmail.com>
> > Cc: QUIC WG <quic@ietf.org>; Dmitri Tikhonov
> > <dtikhonov@litespeedtech.com>
> > Subject: RE: Unidirectional streams PR
> >
> >
> >
> > I agree that looking at the layers of abstraction is useful. In
> > principle having the wire protocol just have constructs for
> > unidirectional streams does not in itself limit creating
> > bi-directional communication flows, supported at either the library or
> application layer.
> >
> >
> >
> > However, there need to be a standard way of doing bi-directional
> > communication for migrating applications implemented using a socket
> > style api. It needs to be easy to move an existing application from TCP
> to QUIC.
> > This move may be attractive in many situations as QUIC gives improved
> > security and may allow greater throughput due to the more modern (and
> > customizable) congestion control algorithms compared to the OS TCP stack.
> >
> >
> >
> > For migrating standard socket api applications I don’t think it would
> > be appropriate to leave the work to the application to do correlation,
> > at least the library should be providing this service using the wire
> > protocol as appropriate. Clearly we want a client written with one
> > library to be able to communicate successfully with a server written
> using a different library.
> > This needs some form of standardization of the signalling. This could
> > either be a building block overlay on top of QUIC, or implemented at
> > the wire protocol level.
> >
> >
> >
> > In terms of patterns I think the following may be some of the most
> > common patterns (with potential to be provided at the library and or
> > wire protocol level).
> >
> > I/O pattern  : Example
> >
> > 1/0   : An input only flow, perhaps a data logger like syslog with no
> > confirmation/feedback
> >
> > 0/1  : an output only flow, perhaps a topic message bus service with
> > no confirmation/feedback
> >
> > 1/1 : standard TCP applications with a single flow per connection
> >
> > 1/* : single input, many output, modelling STDIN/STDOUT+STDERR
> >
> > (1/1)* : multiplexed pairs of flows – supporting multiple sockets
> > muxed onto a single QUIC connection
> >
> >
> >
> > Obviously, an application would always have the option to combine any
> > single direction flows with application level correlators to construct
> > more complex flows if desired.
> >
> >
> >
> > At the moment my gut says the 1/1 use-case is common enough that the
> > wire protocol should provide a standard mechanism to support it as a
> > standard overlay would probably end up being treated as part of the
> > wire format anyway.
> >
> >
> >
> > Perhaps streams should be explicitly created with a CREATE_STREAM
> > frame which would be capable of defining multiple related streams?
> >
> > There is the option of whether only (1/1) pairs can be created this
> > way, or
> > (1/n) combinations could be supported (with an application defined way
> > to identify the use of each of the output streams). A step further may
> > be that there is a transport parameter that defines whether the server
> > is allowed to create additional streams, or if stream creation is
> > purely client driven (like TCP). I don’t know if either of these would
> > simplify how to handle stream accounting, and in particular only
> > creating a flow when all parties have sufficient allowances left.
> >
> >
> >
> > Thomas
> >
> >
> >
> > From: QUIC [mailto:quic-bounces@ietf.org] On Behalf Of Jo Kulik
> > Sent: 28 June 2017 15:09
> > To: Mikkel Fahnøe Jørgensen <mikkelfj@gmail.com>
> > Cc: QUIC WG <quic@ietf.org>; Dmitri Tikhonov
> > <dtikhonov@litespeedtech.com>
> > Subject: Re: Unidirectional streams PR
> >
> >
> >
> > I'd like to pop back up to a comment Igor made last week, because I
> > find it helpful in thinking about the design space:
> >
> >
> >
> > I think of three layers of abstraction:
> >
> > 1.       QUIC Wire Protocol (the thing described by the QUIC Transport
> RFC)
> > 2.       QUIC Library API (a library exposing some useful abstractions --
> > such as blocking/non-blocking unidirectional streams and bidirectional
> > “sockets” -- and implementing them using QUIC Wire Protocol)
> > 3.       Application (something that uses QUIC Library APIs)
> >
> > I think there is some argument to be made that Martin's original
> > proposal did not take into account how we would achieve (2) for
> > bi-directional streams.  (I don't think it strictly said "thou shalt
> > not do (2)" either, but that is up to interpretation.)
> >
> >
> >
> > Several people have argued that we do not want every application to
> > have to re-implement bi-directional streams (3) for every application,
> > and this is not how g-quic (our largest deployment) works right now.
> > These arguments make sense to me, but YMMV.
> >
> >
> >
> > Just because the particular *mechanism* that is being proposed has
> > some issues, however, doesn't scream out to me, at least, that we
> > should abandon this particular *design goal*.  The goal being a
> > transport protocol that can elegantly fit with a uni/bi stream model.
> > Now, if we conclude that there can never be an elegant model that
> > achieves this goal, then so be it.  But I also feel like we haven't
> > reached that point in the discussion yet.  (At the very least, this
> > discussion has been fruitful to me in terms of mapping the design space
> and elucidating requirements).
> >
> >
> >
> > One of the reasons I still think this design goal is under
> > consideration is that Ian and Igor/Mike have been talking about
> > alternate solutions which have a similar flavor.  During the recent
> > "quiet"ness on the thread, personally, I've been waiting to hear more
> from them.
> >
> >
> >
> > On Wed, Jun 28, 2017 at 9:41 AM, Mikkel Fahnøe Jørgensen
> > <mikkelfj@gmail.com> wrote:
> >
> > In reply to Ranjeeth
> >
> >
> >
> > It is not only a matter of simplicity for the sake of simplicity:
> >
> >
> >
> > - A complex transport layer might end up being poorly implemented
> > leading to reduced interoperability and ultimately adoption. This
> > complexity is not only in implementation but also in understanding the
> > exact semantics of stream lifetime. Even if the spec is sufficiently
> > clear, it will still be open to misinterpretations.
> >
> >
> >
> > - Bi-directional state may have to be maintained longer and with more
> > overhead than with uni-directional streams, especially under loss,
> > potentially leading to poor performance and poor resource utilisation
> > because the transport layer has insufficient information.
> >
> >
> >
> > - The extra complexity at the application layer may be overstated - it
> > is significantly simpler to manage a map that associates to two
> > streams than it is to maintain bi-directional state at the transport
> > layer. It is even possible to implicitly link streams with same
> > identifiers, e.g. in a RPC scenario. That said, I do see a potential
> > benefit of a wrapper that implements the common bi-directional case.
> >
> >
> >
> > - Complexity at the application layer may be duplicated, but
> > implementation errors are also isolated to that application.
> > Specifically for HTTP I would assume that QUIC transport and QUIC HTTP
> > implementers would be large the same for a long time to come, so I
> > would not expect the tradeoff here to be particularly concerning.
> >
> >
> >
> > - Unix pipes are traditionally constructed as a pair of
> > uni-directional file descriptors and that is a reasonably proven
> > model. C’s standard library stdin, stdout and stderr is an example of
> > an asymmetric model with implicit linkage between uni-directional file
> descriptors.
> >
> >
> >
> > - There are lots of use cases for non-HTTP like connectivity - Kafka
> > high volume message queuing for example. The industry trend appears to
> > move towards asynchronous processing and messaging. It depends on
> > whether you look at QUIC as a TCP + TLS replacement, or as a HTTPS /
> > REST RPC replacement.
> >
> >
> >
> > - Uni-directional streams may currently be unproven in the wild, but a
> > proposal is needed before an implementation can be made and testet. I
> > agree that it is easy to design into wrong assumptions without real
> world testing.
> >
> >
> >
> > - There will hopefully not be a large number of successors to QUIC -
> > perhaps some purpose specific variants, e.g. for embedded use.
> > Widespread adaptation and compatibility is very necessary so it makes
> > sense to have QUIC being sufficiently simple and expressive to achieve
> > this goal. A polymorf QUIC will not achieve that goal. On the other
> > hand, a solid QUIC foundation can be used for a large number of
> application protocols.
> >
> >
> >
> > - Finally, it may turn out that uni-directional streams just is a bad
> > idea - I doubt it, but I do believe real world tests are needed.
> >
> >
> >
> > Kind Regards,
> >
> > Mikkel Fahnøe Jørgensen
> >
> >
> >
> > On 28 June 2017 at 14.42.36, Dmitri Tikhonov
> > (dtikhonov@litespeedtech.com)
> > wrote:
> >
> > On Tue, Jun 27, 2017 at 02:31:38PM -0700, Ranjeeth Kumar Dasineni wrote:
> >> 2. We are overplaying the simplicity of design. Even if we deem
> >> deployment experience not a concern, if every application layer
> >> protocol that needs support for bidirectional streams has to
> >> implement some correlators and such above, that's a net negative in
> terms of complexity.
> >
> > This is an important point: we want QUIC adoption to be made easy.
> > A program that speaks HTTP today should be able to use an existing
> > QUIC library without having to emulate bidirectional streams in order
> > to fit it into HTTP usage pattern. Forcing every one of these programs
> > to do this is certainly a hurdle.
> >
> > - Dmitri.
> >
> >
>
>