Re: [art] Artart last call review of draft-ietf-alto-cost-mode-02

[Francesca Palombini <[e-mail]>]

Jaime: many thanks for this review, and thanks to the authors for addressing Jaime’s comments. I have balloted No Objection on this doc.

Francesca

From: art <[e-mail]> on behalf of [e-mail] <[e-mail]>
Date: Thursday, 12 May 2022 at 14:32
To: Jaime Jimenez <[e-mail]>, [e-mail] <[e-mail]>
Cc: [e-mail] <[e-mail]>, [e-mail] <[e-mail]>, [e-mail] <[e-mail]>
Subject: Re: [art] Artart last call review of draft-ietf-alto-cost-mode-02
Hi Jaime,

Thank you for the careful review.

Please see inline.

Cheers,
Med

> -----Message d'origine-----
> De : Jaime Jimenez via Datatracker <[e-mail]>
> Envoyé : mercredi 11 mai 2022 10:01
> À : [e-mail]
> Cc : [e-mail]; [e-mail]; last-
> [e-mail]
> Objet : Artart last call review of draft-ietf-alto-cost-mode-02
>
> Reviewer: Jaime Jimenez
> Review result: Ready with Nits
>
> Dear all,
>
> I am the assigned reviewer for the Applications and Real Time Area
> Review Team (ART-ART).
>
> Nits/editorial comments:
>
> One minor comment is that since this document sets a registry with
> a limited set of cost-modes (for now "numerical" and "ordinal")
> that require IANA registration anyways, why not assign some 8-bit
> codes ("0x0, 0x1") to save space on the wire.?

[Med] We don't consider that because that would require changes to existing 7285 implementations.

Probably this has
> already been discussed somewhere in the WG, so I apologize if I am
> way off on this.
>
> Another minor comment is that RFC7285 defines variations of a JSON
> they use in now few different locations in the document, section
> 10.1, section 10.6, section 10.8.2 and after this document is
> approved also on section 10.5. To make transparent to the that
> they are different I would consider writing a line on section 3.2
> saying that neither the '.' separator nor the the at sign ('@')
> are allowed.

[Med] I don't think such a note is needed as the text is clear about allowed characters. Thanks.

>
> There is an extra space on 3.2: U+0041 -U+005A Should be: U+0041-
> U+005A
>

[Med] Fixed.

> There is a typo on 3.2: ('_', +005F)
> Should be: ('_', U+005F)
>

[Med] Fixed


_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.

_______________________________________________
art mailing list
[e-mail]
https://www.ietf.org/mailman/listinfo/art