Re: [alto] Chair review of draft-ietf-alto-incr-update-sse-17

Vijay Gurbani <vijay.gurbani@gmail.com> Mon, 06 January 2020 21:34 UTC

Return-Path: <vijay.gurbani@gmail.com>
X-Original-To: alto@ietfa.amsl.com
Delivered-To: alto@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3398A12007A; Mon, 6 Jan 2020 13:34:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.997
X-Spam-Level:
X-Spam-Status: No, score=-1.997 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=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=gmail.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 JeUrWvsv7foN; Mon, 6 Jan 2020 13:33:58 -0800 (PST)
Received: from mail-ed1-x529.google.com (mail-ed1-x529.google.com [IPv6:2a00:1450:4864:20::529]) (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 AC9541200CE; Mon, 6 Jan 2020 13:33:57 -0800 (PST)
Received: by mail-ed1-x529.google.com with SMTP id l8so48690398edw.1; Mon, 06 Jan 2020 13:33:57 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=FCMWPuJGPDszWqOa1BkL60Tb2cwBDCrr8eh2AzGTdN8=; b=n4slx6NlgtGPDk3USYS6Sr+C8G3jHS0be3sZZp/vD7KHRb9Fdo/WnTt8n2fNyLBVWF ENTVf6tdzYWAR3dJZCynVElrZHh6KnCJY3G42NkGBuh8wbICnKfEON/9mH7/nEfrHJlI yBLM+xWgyLumRyZby/SoA984fc4Qqlytr1hcyuBkVFxMYjniBTt6t2o7VVR9sNf8q+x1 +oCY61M6ex0kzgW9JYlqeYR53tviHhqo7kc6Dks7xd5Deoau7vTPn9zO3m2nxaWyATJg njH+lpg4ot8hRy0EGryCbBWZkppMc7+uJYxG7q3WTLoc84MTehXUh2dMW/0aeXeGIh+G IMlQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=FCMWPuJGPDszWqOa1BkL60Tb2cwBDCrr8eh2AzGTdN8=; b=RoTro5lay9pF3gkcBL2NWlM6S3gCAqXPhVOd8HjfyzJ27W+vGXcxDqM18QPgoF32Uv zBMeRvHKFrbwoa5NxNcANwbVGZqgvEqa20L1SFXNCJJR07RBunrpZnAU12Xrg3OQGOGw QnqiNSIRJWS0OWXriYqwiJXsOU0oUSaLmwszwQg8c/YiRfSKZAy/kY3c+9uodsHxv2lV 6R/ljzzwgykws8F1Vg/UuOxvzYcPmbBTygMeuc/nS63ZiJKvdgcPUFYzRSinq+4Ksruz e9Y2i1vQNbPra64dTnAFrXse7a/U6iAPjBhl1jSWn9J9vaLqlZPT9NcbV0DPfeNj2q8J kPqw==
X-Gm-Message-State: APjAAAWtYkpjNJRph3Pna4SsbxYiQK58o7qbBfMcir/QwfJYT9FYPPPL 7X3Kt7OepfaULP+Jc67ptLckMQTY/67LuUXW/VE=
X-Google-Smtp-Source: APXvYqxVe1AFaIb3RHjVtlNf3wOkGg/lVOsrti9vEGxY+1bOBIO+BMW65Wdpjsgd5/TtisHYX5Iv3RYFBrt9X/h2o88=
X-Received: by 2002:a17:906:c78b:: with SMTP id cw11mr112041627ejb.262.1578346436112; Mon, 06 Jan 2020 13:33:56 -0800 (PST)
MIME-Version: 1.0
References: <CAMMTW_KOHEfE7AojeviUdUVqmDLmJQ97+hUeoMKj1A0wwAzY6w@mail.gmail.com> <CAEDarXKPD7x35GRFiYgMy2jnq5NsfYumDR4K5DbQz_AhJHKeig@mail.gmail.com> <2a1f25f3-65ca-4343-a0a5-0d387840ff19@inet.tu-berlin.de>
In-Reply-To: <2a1f25f3-65ca-4343-a0a5-0d387840ff19@inet.tu-berlin.de>
From: Vijay Gurbani <vijay.gurbani@gmail.com>
Date: Mon, 6 Jan 2020 15:35:28 -0600
Message-ID: <CAMMTW_JhJw65fczkEVBaQ3fRfn+JzNbZQWjcfP1eavOWHgRn3w@mail.gmail.com>
To: Ingmar Poese <ingmar@inet.tu-berlin.de>
Cc: Danny Alex Lachos Perez <dlachosper@gmail.com>, draft-ietf-alto-incr-update-sse@ietf.org, IETF ALTO <alto@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000a83a43059b7f6abf"
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/8WXUvXvdWZ4rukmayUOebvcfOME>
Subject: Re: [alto] Chair review of draft-ietf-alto-incr-update-sse-17
X-BeenThere: alto@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Application-Layer Traffic Optimization \(alto\) WG mailing list" <alto.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/alto>, <mailto:alto-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/alto/>
List-Post: <mailto:alto@ietf.org>
List-Help: <mailto:alto-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/alto>, <mailto:alto-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 06 Jan 2020 21:34:00 -0000

Dear Ingmar: This is great information, thanks.
Did you implement both JSON patch and JSON merge patch?
Thanks,
- vijay

On Mon, Jan 6, 2020 at 2:50 PM Ingmar Poese <ingmar@inet.tu-berlin.de>
wrote:

> Hi Danny,
>
> We (BENOCS) have implemented the Alto SSE, but are lacking a partner to
> speak it to in production scale.
>
> In our testing environment it seems to work fine (even with production
> data).
>
> I was unable to attend Singapore, but will be available in vancouver (and
> possibly madrid) to chat/present the research.
>
> Best,
> Ingmar
>
> Am 6. Jan. 2020, 19:56, um 19:56, Danny Alex Lachos Perez <
> dlachosper@gmail.com> schrieb:
> >Hello Vijay,
> >Happy new year!!!
> >
> >Just a quick comment to your question about implementations of ALTO-SSE
> >
> >There is a related work "Steering Hyper-Giants’ Traffic at Scale" [0]
> >where
> >ALTO is used as a northbound interface in a *real operational
> >environment
> >at scale*.
> >The authors mention the SSE extension (but I am not sure if this
> >extension
> >was also tested).
> >
> >Best regards,
> >
> >Danny Lachos
> >
> >[0]
> >https://mailarchive.ietf.org/arch/msg/alto/h7QJRu47NbTvfcnW2fveFqCBRdw
> >
> >On Mon, Jan 6, 2020 at 4:32 PM Vijay Gurbani <vijay.gurbani@gmail.com>
> >wrote:
> >
> >> All: Happy new year.
> >>
> >> In preparation of moving alto-incr-update-sse ahead, I have performed
> >a
> >> chair review of the work.  Overall, the document is well written,
> >mature,
> >> and considers various design tradeoffs.  This is fairly mature work,
> >and we
> >> should move it out of the WG following the resolution to the review
> >below
> >> and an additional review by Jensen Zhang [1].
> >>
> >> --- Begin chair review
> >>
> >> I am curious --- are there any known implementations of alto-sse?
> >>
> >> MAJOR
> >> -S10.1: This is an important discussion.  However, this discussion is
> >> written primarily from a viewpoint of an ALTO client, but if I
> >understand
> >> it correctly, it should be written from the viewpoint of an ALTO
> >stream
> >> server since it is the stream server that is generating the event
> >since
> >> that is the source that should be told to behave conservatively.
> >Should
> >> this section be re-written to exhort the stream server to send out
> >full
> >> cost maps in chunked format, where each chunk is at most 2,000
> >octets?
> >> That way, the clients are not overwhelmed.  Thoughts?
> >>
> >> MINOR
> >> S3: It is rather unfortunate that one of the services is named
> >“Stream
> >> Control Service” as this may be conflated by the uninitiated reader
> >with
> >> the Stream Control Transmission Protocol (SCTP) service, a transport
> >layer
> >> protocol.  Clearly, that is not the intent here.  However, I am
> >loathe to
> >> suggest a new naming scheme this late in the document publication
> >phase, so
> >> perhaps the best we can do now is to add a note explicitly
> >disassociating
> >> Stream Control Service of ALTO from SCTP.  Perhaps something like:
> >s/from
> >> the update stream./from the update stream. (Note that the Stream
> >Control
> >> Service in ALTO has no association with the similarly named Stream
> >Control
> >> Transmission Protocol [RFC4960].)/
> >>
> >> S4: The phrase “Using existing techniques wherever possible,” implies
> >that
> >> you have used other, perhaps new techniques at other places.  Is that
> >the
> >> case?  If so, please enumerate the new techniques; if not, perhaps
> >reword
> >> as s/Using existing techniques wherever possible,/Using existing
> >> techniques,/
> >>
> >> -S4.2.1: “This document adopts the JSON merge patch message format to
> >> encode incremental changes, but uses a different transport
> >mechanism.” ==>
> >> Not sure how to interpret this.  Since alto-sse uses the HTTP PATCH
> >method
> >> to affect incremental updates, it uses the same “transport mechanism”
> >> (i.e., TLS).  Perhaps you meant “...., but uses a different HTTP
> >method,
> >> i.e., it uses POST instead of PATCH (details in Section 5).”?
> >>
> >> -S4.2.1, page 10: s/, and (3) assigns a new tag to the network map:/,
> >(3)
> >> leaves “PID3” unmodified, and (4) assigns a new tag to the network
> >map:/
> >>
> >> -S6.1: Is there some magic about the numbers “1” and “2” assigned to
> >> substream IDs?  In other words, must substream IDs begin with 1 and
> >> monotonically increase?  If so, state that.  If not, then state that
> >> substream IDs must begin with a random number between [1, 10] and
> >> monotonically increase from there on for each new substream.  That
> >is, if
> >> the first substream ID is 6, then subsequent substream IDs from the
> >client
> >> should monotonically increase from this starting value.  (I will let
> >the
> >> protocol designers come up with the exact text to impart this.)
> >>
> >> NITS
> >> -S5, page 16: s/this design allows/this document allows/
> >> (Overworked use of “design”: “...flexible protocol design, this
> >design…”).
> >>
> >> -S10.1: s/single character array./character array./
> >>
> >> -S10.1: s/client computer/client/
> >>
> >> --- End of chair review
> >>
> >> Additionally, the work has also been reviewed by Jensen [1].
> >>
> >> Authors, please attend to the comments indicated in this review and
> >> Jensen's review and release a new version in order to move the work
> >forward.
> >>
> >> [1]
> >https://mailarchive.ietf.org/arch/msg/alto/C9_tS44bz7kq84Z3cpZZkMeUDFc
> >>
> >> Thank you.
> >>
> >> - vijay
> >> _______________________________________________
> >> alto mailing list
> >> alto@ietf.org
> >> https://www.ietf.org/mailman/listinfo/alto
> >>
> >
> >
> >------------------------------------------------------------------------
> >
> >_______________________________________________
> >alto mailing list
> >alto@ietf.org
> >https://www.ietf.org/mailman/listinfo/alto
>
>