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

Hi All,

+1 on keeping data items unordered.  We worked hard on core DLEP to
ensure that there was no ordering constraint, and it would be a pity to
introduce it in an extension.

Cheers,

Rick

On Tue, 2018-06-12 at 17:52 -0400, Lou Berger wrote:
> 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-0
> 3-comments
> and full text can be seen at:
> https://xml2rfc.tools.ietf.org/cgi-bin/xml2rfc.cgi?url=https://raw.gi
> thubusercontent.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
> > manet@ietf.org
> > https://www.ietf.org/mailman/listinfo/manet
> 
> _______________________________________________
> manet mailing list
> manet@ietf.org
> https://www.ietf.org/mailman/listinfo/manet