Re: [Gen-art] Gen-ART Last Call review of draft-ietf-cdni-metadata-18

Kevin Ma J <kevin.j.ma@ericsson.com> Tue, 28 June 2016 16:38 UTC

Return-Path: <kevin.j.ma@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C937E12B047 for <gen-art@ietfa.amsl.com>; Tue, 28 Jun 2016 09:38:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 Z0GS4KFO0nkt for <gen-art@ietfa.amsl.com>; Tue, 28 Jun 2016 09:38:25 -0700 (PDT)
Received: from usplmg21.ericsson.net (usplmg21.ericsson.net [198.24.6.65]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0FE6C12B03F for <gen-art@ietf.org>; Tue, 28 Jun 2016 09:38:25 -0700 (PDT)
X-AuditID: c6180641-f796f6d000000e1e-47-5772a7c0d237
Received: from EUSAAHC003.ericsson.se (Unknown_Domain [147.117.188.81]) by usplmg21.ericsson.net (Symantec Mail Security) with SMTP id BF.B9.03614.0C7A2775; Tue, 28 Jun 2016 18:37:21 +0200 (CEST)
Received: from EUSAAMB103.ericsson.se ([147.117.188.120]) by EUSAAHC003.ericsson.se ([147.117.188.81]) with mapi id 14.03.0294.000; Tue, 28 Jun 2016 12:38:23 -0400
From: Kevin Ma J <kevin.j.ma@ericsson.com>
To: Meral Shirazipour <meral.shirazipour@ericsson.com>, "draft-ietf-cdni-metadata.all@tools.ietf.org" <draft-ietf-cdni-metadata.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART Last Call review of draft-ietf-cdni-metadata-18
Thread-Index: AdHQzk1P1TOJ+3GbR4a0b74/79fmOwAjMnfg
Date: Tue, 28 Jun 2016 16:38:23 +0000
Message-ID: <A419F67F880AB2468214E154CB8A556206E3855F@eusaamb103.ericsson.se>
References: <ABCAA4EF18F17B4FB619EA93DEF7939A4E65B272@eusaamb107.ericsson.se>
In-Reply-To: <ABCAA4EF18F17B4FB619EA93DEF7939A4E65B272@eusaamb107.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.9]
Content-Type: multipart/alternative; boundary="_000_A419F67F880AB2468214E154CB8A556206E3855Feusaamb103erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpjkeLIzCtJLcpLzFFi42KZXLonUPfg8qJwg5ZH7BbnLixls7j66jOL A5PHkiU/mTy+XP7MFsAUxWWTkpqTWZZapG+XwJXx8+NL9oJVcRWNf+axNzBuCe5i5OSQEDCR WPC6mQ3CFpO4cG89kM3FISRwlFHi+bo3LBDOckaJVZ/aWUGq2AS0JB5//csEkhAROMAo0XK9 k72LkYNDWMBFonkbB0iNiICrxOaNH9ggbCOJ6dsPM4PYLAKqEn1fdoDN4RXwlWjZdpQRxBYC ss/MfwFWwyngJ3F52yGwOCPQRd9PrWECsZkFxCVuPZnPBHGpgMSSPeeZIWxRiZeP/7FC2IoS +/qng53DLJAvsfqEJsQqQYmTM5+wTGAUmYVk0iyEqllIqiBKdCQW7P7EBmFrSyxb+JoZxj5z 4DETsvgCRvZVjBylxQU5uelGhpsYgbFzTILNcQfj3l7PQ4wCHIxKPLwPFhaGC7EmlhVX5h5i lOBgVhLhzVtSFC7Em5JYWZValB9fVJqTWnyIUZqDRUmcV/+lYriQQHpiSWp2ampBahFMlomD U6qBkUPTdf4lGfU65RXTt5ycu5ZRc99Tzoz/jSbLp+/ce3tLhJVRfZda4dWZWv8dDU3PVU7Z 9mPnfLG5hYtksmys3Q+WSGyb8SH+1kzRhW+Xld3QMqpyUO/uOaq3fM2GmWLL7BnPv9+0Kt7e MUgo5N8MhusxzxuXrV1kznNogdrWhQpbtnSqpUgvt1BiKc5INNRiLipOBAAg+2ZvmQIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/2pG8NZX2e1ItiJwWGxYWpAxm3dI>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-ietf-cdni-metadata-18
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 28 Jun 2016 16:38:28 -0000

Hi Meral,

  Thanks for the review.  We will fix the typos in the next revision.

> Question: The $ character is not described. What does $$ mean if $? is the one character match?

  The "$" is the escape character, so "$$" allows the use of an actual dollar sign in the text.  Will add clarification text.

> General, does the "MI" prefix stand for "Metadata Interface"?

  Correct, MI stands for Metadata Interface.  The convention has been to prefix the payload types with the interface abbreviation.

thanx!

--  Kevin J. Ma

From: Meral Shirazipour [mailto:meral.shirazipour@ericsson.com]
Sent: Monday, June 27, 2016 7:49 PM
To: draft-ietf-cdni-metadata.all@tools.ietf.org; gen-art@ietf.org
Subject: Gen-ART Last Call review of draft-ietf-cdni-metadata-18

I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair.  Please treat these comments just like any other last call comments.
For more information, please see the FAQ at http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq .


Document: draft-ietf-cdni-metadata-18
Reviewer: Meral Shirazipour
Review Date: 2016-06-27
IETF LC End Date:  2016-06-27
IESG Telechat date: 2016-07-07


Summary:
This draft is ready to be published as Standards Track RFC but I have some comments.

Major issues:

Minor issues:


Nits/editorial comments:
-[Page 19]:
"
Property: pattern
         Description: A pattern for string matching.  The pattern can
         contain the wildcards * and ?, where * matches any sequence of
         characters (including the empty string) and ? matches exactly
         one character.  The three literals $, * and ? should be escaped
         as $$, $* and $?. All other characters are treated as literals.
"

Question: The $ character is not described. What does $$ mean if $? is the one character match?


-[Page 20], section 4.1.6,
"to evaulate"--typo-->"to evaluate"
"is specifed"--typo-->"is specified"

-[Page 24], section 4.2.1.1
"objecct,"--typo-->"object,"

-[Page 50-54], sections 7.1.1 to 7.1.20
"capabilitiy"--typo?-->"capability" (many times)

-General, does the "MI" prefix stand for "Metadata Interface"?

Best Regards,
Meral
---
Meral Shirazipour
Ericsson Research
www.ericsson.com<http://www.ericsson.com>