Re: [manet] ordered data items in DLEP pause extension

[Lou Berger <[e-mail]>]

Hi,

David and I discussed this - as I didn't feel strongly about this and 
his comment was based on actual implementation experience, we've put 
together a version that addresses his comment.  We also clarified the 
sub data type field definition as it was somewhat under defined in the 
previous version (I views this to be not a technical change, but rather 
an editorial clarification) .

The changes are:
    1) make the Queue Parameter Sub Data Item unordered by adding a 
Queue Index field to the sub-DI
    2) formally define  Sub Data Item Type, the value (of 1) is not changed.

The detailed diff can be seen at:
https://github.com/louberger/dlep-extensions/compare/working/lb/rev-03-comments
and full text can be seen at:
https://xml2rfc.tools.ietf.org/cgi-bin/xml2rfc.cgi?url=https://raw.githubusercontent.com/louberger/dlep-extensions/master/pause/draft-ietf-manet-dlep-pause-extension.xml

*No that this document is post LC*

Does anyone object to this change?

Chairs,
     Please let us know if / when we should submit this rev.

Thank you,
Lou (and David)

PS we'll update the flow control document to have the same clarification 
(#2 above) once we publish this change.

On 6/11/2018 12:56 PM, Wiggins, David - 0665 - MITLL wrote:
>
> In draft-ietf-manet-dlep-pause-extension-03, section 3.1.1 says “Queue 
> Parameter Sub Data Items are an ordered list composed of sub data 
> items with a common format.    The first sub data item is assigned a 
> Queue Index value of 1, and subsequent data items are numbered 
> incrementally.”
>
> I’d rather not allow the order of data items, even though they are sub 
> data items, to convey any meaning.  It isn’t done anywhere else in 
> DLEP (as far as I know), and I think that’s a good thing.  It allows a 
> wider variety of implementations.  I believe this is easily fixed by 
> adding a queue index field to the Queue Parameter Sub Data Item and 
> adjusting the above-quoted sentences.
>
> David
>
>
>
> _______________________________________________
> manet mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/manet