Re: [alto] Reviews on draft-gao-alto-fcs-02

Shenshen Chen <cs90911@gmail.com> Fri, 30 June 2017 13:20 UTC

Return-Path: <cs90911@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 5B37E12955F for <alto@ietfa.amsl.com>; Fri, 30 Jun 2017 06:20:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.449
X-Spam-Level:
X-Spam-Status: No, score=-2.449 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-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 nBq9ZxA748Nx for <alto@ietfa.amsl.com>; Fri, 30 Jun 2017 06:20:34 -0700 (PDT)
Received: from mail-vk0-x230.google.com (mail-vk0-x230.google.com [IPv6:2607:f8b0:400c:c05::230]) (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 C05CC1205F0 for <alto@ietf.org>; Fri, 30 Jun 2017 06:20:33 -0700 (PDT)
Received: by mail-vk0-x230.google.com with SMTP id 191so65849876vko.2 for <alto@ietf.org>; Fri, 30 Jun 2017 06:20:33 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=Du9EQb33WtDfVYlL+wOstkQFOgh1A2YaahtnlAgmfo4=; b=TqNAuriTxumdIFE5P8pqJ94FAqjAmSi4/IsjfN3PcqVvZHud/1/wvMxG41MxUo6on8 77aF4YtzVI8frAH9irbM67g8s3LoHt+sRbbgJXpCNimSB5bUgnVgn9I88Fs+Y9J+1PwQ aD7nSrLOtaumLrTjXy/ZBa5J5vD1BKLzCaivso4QeQq3vXSjCck8TjVBNL8bgMVFPsP1 ABh1zGh5LR2voXKlKDNw10HtDHSWc8WEN0shHRUSWHdQwxJBEchisqFuYj4UI74mernd 1/c0PCrmFDzk/WaD9YZFUVyLumEKOJBDnUFgDqOWUkBEFIzq7KYnHyJsnp7QeyAgULnO xaEg==
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=Du9EQb33WtDfVYlL+wOstkQFOgh1A2YaahtnlAgmfo4=; b=QRtYX1LBEDMCk8PD01ZCwJEnI8UR1ZLnoB8uZ7n9GkTXsheDmPwk+w97fsfyLBHvup jIHW4K0XrPa3Bf2LX3EaWcQpAsbsBkr6GNaSdBhSg18lzIK7IjKJIOemKdpivCa3l4H5 Cgj9Ba5suk6smPkDgi74q4GFe8w5qH2FxPW2XbUpd4ao6Ugr2c9WtgBn1gFmxHoC6vIZ 4gnjxueEt01x+lPU5tPjaEaipl8eaaBSh2Mf+kL44RqBqbQbNYtSxXBfQLLA6rz6kj0+ sogHnoyo69tjsshreAAx6aCroFDiWGfP7f3+ThIyS3nJQlOnn+mojghr0iP9Avk/MH15 oeSg==
X-Gm-Message-State: AKS2vOy2Im6H9NmGot0YX8mm2UKU9ooiQFioqeuGgYhVzjtT9LKq2oTc 7VtFKKJCiuwjK1erUshmXHlAevZicw==
X-Received: by 10.31.53.4 with SMTP id c4mr11647179vka.12.1498828832792; Fri, 30 Jun 2017 06:20:32 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.31.236.199 with HTTP; Fri, 30 Jun 2017 06:19:52 -0700 (PDT)
In-Reply-To: <CAAbpuyqZF5ZzOm1bwXFYuSscFAK2sEHcimNyY3AsBZ8Ok49Baw@mail.gmail.com>
References: <07e102bd-999a-c6b3-de5c-e5f01bc5b924@mails.tsinghua.edu.cn> <CAN-YqX2=qFJOyHAVKSGgzJJjGpuJpca2Kht6w0+bPKNQi2cW5g@mail.gmail.com> <CAAbpuyqZF5ZzOm1bwXFYuSscFAK2sEHcimNyY3AsBZ8Ok49Baw@mail.gmail.com>
From: Shenshen Chen <cs90911@gmail.com>
Date: Fri, 30 Jun 2017 21:19:52 +0800
Message-ID: <CAKRjQSC4taLhJ=PRdQrcwSXWm0YY5eWtuMsoxYSMvWkjzeQHnQ@mail.gmail.com>
To: Jensen Zhang <jingxuan.n.zhang@gmail.com>, dennis.yht@gmail.com, gaok12@mails.tsinghua.edu.cn
Cc: alto@ietf.org
Content-Type: multipart/alternative; boundary="001a114476302865df05532d4887"
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/AjpbjWDhWb6RjDeZTxr2wcpuCQA>
Subject: Re: [alto] Reviews on draft-gao-alto-fcs-02
X-BeenThere: alto@ietf.org
X-Mailman-Version: 2.1.22
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: Fri, 30 Jun 2017 13:20:36 -0000

Hi all,

I'd like to join this discussion.

About the flow-cost-confidences, I prefer the original design.
The mean of cost makes sense (the expected cost if pick path randomly)
and can't be deduced from the range (since it may not be an
equal probability distribution).


Here is a list of typos (comments are marked with ">>>"):

[Page 1]
     3.1.  Flow-based Filtered Cost Map  . . . . . . . . . . . . . .   4
       3.1.1.  Capabilities  . . . . . . . . . . . . . . . . . . . .   4
       3.1.2.  Accept Input Parameters . . . . . . . . . . . . . . .   5

>>> Accept => Acceptable, also see 3.3.2. and 4.2.3.

[Page 1]
     3.4.  Example . . . . . . . . . . . . . . . . . . . . . . . . .   8

>>> Example => Examples

>>> Also, the catalog doesn't include Section 4.2.5.1., 4.2.6.1., etc,
which looks weird.

[Page 1]
   The emergence of new networking datapath capabilities has
   substantially increased the flexibility of networking.  For example,
   OpenFlow has emerged as a major southbound protocol for Software-
   Defined Networking, and OpenFlow allows flexible routing beyond
   simple destination-based routing.  In this document, we define a new
   extention to ALTO, namely the Flow Cost Service, for ALTO clients in

>>> extention => extension

   an OpenFlow-enabled network to query ALTO network information.

[Page 6]
   The Protocol field is REQUIRED.  The available values contain
   different protocols including layer two protocols (e.g.  "eth") and
   layer three protocols (e.g.  "ipv4", "ipv6").  It can also be
   specified upper-layer protocols (e.g. "udp", "tcp", "ssh", "http" and
   "ftp").  The source and destination protocols MUST NOT be conflict.

>>> be conflict => be conflicted

   In every EndpointFilter object of either "endpoints" field or
   "endpoint-flows" field, if the source protocol is conflict with the

>>> is conflict with => has conflict with

   destination protocol, this endpoint pair is invalid.  For different
   protocols, some additional constraints are defined in Section 3.2.3.

[Page 7]
   protocols:  Defines a list of JSONString indicating the supported
      Protocol values of the EndpointURI in the request.  The ALTO
      server does not have to claim "ipv4" and "ipv6" in this field
      explictly, because they are supported by default.  If not present,

>>> explictly => explicitly

      this field MUST be interpreted as if it is specified the default
      supported protocols "ipv4" and "ipv6".

[Page 16]
   The ALTO servers can provide more information to the clients when
   requests have errors.  The FlowCostErrorMap below can provide basic
   information about two most common errors for the flow cost service.
   The ALTO servers MAY include it as the data component of an ALTO
   error response.  If multiple errors are identified, the ALTO server
   MUST return exactly one error code according to Section 8.5.2 of
   [RFC7285] .

[Page 20]
  Security considerations:  Security considerations are identical to
      those specified in Section 15 of [RFC7285] .

>>> remove the redundent space between [RFC7285] and full stop

[Page 16, 17]
   Some header fields may have conflicts.  For example, IPv4 fields and
   IPv6 fields can never appear in the same packet, nor can TCP and UDP
   ports.  These header fields MUST not be included in the same flow
   filter, otherwise the ALTO server MUST return an ALTO error response,
   with the error code "E_INVALID_FIELD_VALUE".  As specified in

>>> , otherwise => . Otherwise,
    remove the comma before "with"

   Section 8.5.2 of [RFC7285], the ALTO server MAY include the "field"
   and the "value" in the "meta" field.  In this case, the ALTO server
   MUST use the flow ID as the "field" and the flow filter as the
   "value".  However, the recommended approach is to use the
   FlowCostErrorMap, where the server CAN provide the conflicting typed
   header fields in the "conflicts" field of the FlowFilterError
   associated with the corresponding flow ID.

Best Regards,
Shenshen