Dear WG and authors of draft-ietf-alto-cost-mode,

I am posting this review of draft-ietf-alto-cost-mode-01 to the mailing list, as
part of my shepherd write-up. Any comments and feedback are more than welcome!

Best,
Kai

===================

This draft extends the base ALTO protocol (RFC 7285) by relaxing the constraint
on valid cost mode values and introducing a new IANA (sub-)registry to document
new cost mode values. The motivation is clear and the proposed mechanism
is clean. Most comments raised in Call for Adoption and WGLC are addressed in
the latest revision except Dhruv's comment [1] on giving more detailed
specifications of the contents in IANA registry. There are two remaining
comments and I think the draft is ready for publication once they are addressed.

Comments:

Section 3.1, last paragraph: The paragraph says 

   Future documents that define a new cost mode SHOULD indicate whether
   that new cost mode applies to all or a subset of cost metrics.

In that case, it seems to me that the default behavior should be specified in 
case
the applicability of the new cost mode is not indicated. Either the "SHOULD" 
keyword
is replaced by "MUST" or an additional sentence is required, e.g.,

NEW:
    If not explicitly specified, the new cost mode applies to all cost metrics.

Section 4:

I also agree with Dhruv's comment that the contents of the "ALTO Cost Modes"
registry should be better specified. While the initial entries set good examples
of how to register a new cost mode, it can still be helpful if the format and
content of each field are specified in more details, e.g., using similar
specifications in Sections 14.2 and 14.3 of RFC 7285 (as suggested by Dhruv).

I also suggest renaming the "Specification" field to "Intended Semantics", to be
consistent with other ALTO registries (in RFC 7285 and in the unified property 
draft).

[1] https://mailarchive.ietf.org/arch/msg/alto/B1agkfVtdu7tsad2-MzErQXMk44/
_______________________________________________
alto mailing list
alto@ietf.org
https://www.ietf.org/mailman/listinfo/alto

Reply via email to