- From: Greg Wilkins <gregw@intalio.com>
- Date: Tue, 8 Jul 2014 14:12:40 +1000
- To: Mark Nottingham <mnot@mnot.net>
- Cc: HTTP Working Group <ietf-http-wg@w3.org>
- Message-ID: <CAH_y2NERZ4uzPqemNMgmPpytGOvuvTDQd-Dg5Qoc1mjADxM9qQ@mail.gmail.com>
All,
Here is my summary of objections to this proposal so far.
>From my biased point of view, the following objections have been raised but
adequately responded to:
- Too late in the process
- It is too late when the chair tells us it is too late
- The lack of ability to tune frame size has not yet definitely been
identified/accepted as an issue.
- real use-cases have been provided where a 16KB is far from optimal.
- I don't like jumbo frames
- I don't like continuation - deadlock!
- Could achieve the same with a prepended size to a
HEADERS+CONTINUATION* sequence
- Worst of both worlds
- Doesn't solve all DOS attacks
- never claimed it did
- It is too complex
- Perhaps WINDOW_UPDATE, but the core change is very simple.
But I do think we need to discuss these ones some more:
- The WINDOW_UPDATE change is adding a feature
- The WINDOW_UPDATE change is a per stream setting - thus it is bad.
- The mechanism is intended to be part of the flow control
algorithm. It is the intention of http2 to not specify a flow control
algorithm, but instead define the bounds of flow control and to then let
implementations develop better algorithms over time, with experience and
potentially for special purposes. Adding frame size as a variable that
flow control algorithms can adjust will allow for more algorithms to be
explored without requirements to change the specification or deploy an
extensions. However, this part of the proposal is indeed separable from
the rest of it.
- It assumes 'bigger is better' and will make for a poor QoS.
- It is definitely not the intention of this proposal to just allow
bigger frames, rather to move the frame size limit from a fixed
capacity to
a configurable parameter. In some circumstances it may well be that the
frame size is configured smaller rather than larger. I think
it is really
important that we explore this issue an make sure that the proposal does
not allow inadvertent nor deliberate degradation of QoS and
while i believe
this is already the case, we need to ensure that it is understood.
- It prevents streaming headers
- yes it does (no such thing as a free lunch!) However, other
limitations of hpack have already severely limited the practicality of
streaming headers. In fact I've not yet seen a good description of a use
case where a header can be safely streamed without risking HOL
blocking.
I believe that the benefits of this proposal outweighs this particular
limitation. If streaming headers really are a hard requirement, then we
should look to HPACK to make changes, not the framing layer.
Have I missed any?
cheers
On 8 July 2014 13:39, Greg Wilkins <gregw@intalio.com> wrote:
> I've pushed some fixes to a few editorial issues in the proposal
> identified by Martin
>
>
> https://github.com/gregw/http2-spec/commit/b55765fc173823da1e515ec653604d9cea6ce820
>
> cheers
>
>
>
> On 8 July 2014 13:23, Greg Wilkins <gregw@intalio.com> wrote:
>
>>
>> On 8 July 2014 10:58, Mark Nottingham <mnot@mnot.net> wrote:
>>
>>> That's a design decision that was made a long time ago. To reconsider
>>> it, I need more than vague concerns that amount to "I don't like it"; I
>>> need concrete problems that it causes.
>>
>>
>> insert Michael Sweets use-case here.
>>
>> cheers
>>
>>
>>
>> --
>> Greg Wilkins <gregw@intalio.com>
>> http://eclipse.org/jetty HTTP, SPDY, Websocket server and client that
>> scales
>> http://www.webtide.com advice and support for jetty and cometd.
>>
>
>
>
> --
> Greg Wilkins <gregw@intalio.com>
> http://eclipse.org/jetty HTTP, SPDY, Websocket server and client that
> scales
> http://www.webtide.com advice and support for jetty and cometd.
>
--
Greg Wilkins <gregw@intalio.com>
http://eclipse.org/jetty HTTP, SPDY, Websocket server and client that scales
http://www.webtide.com advice and support for jetty and cometd.
Received on Tuesday, 8 July 2014 04:13:10 UTC