[alto] Artart last call review of draft-ietf-alto-unified-props-new-18

2021-09-07 Thread Spencer Dawkins via Datatracker
Reviewer: Spencer Dawkins Review result: Ready with Issues I'm sorry for running late on this review, and please don't be concerned about the length - it includes a lot of draft text as part of the comments. Do The Right Thing, of course. In this text, At first, a map of endpoint properties

Re: [alto] Artart last call review of draft-ietf-alto-cdni-request-routing-alto-16

2021-09-07 Thread Thomas Fossati
Hi Jensen, Thanks very much for addressing my review. Your proposed changes generally look very good to me. Some more (very minor) comments in-lined below. cheers! On 07/09/2021, 12:04, "Jensen Zhang" wrote: > > On Sat, Aug 28, 2021 at 8:28 AM Thomas Fossati via Datatracker > > wrote: > >

Re: [alto] Genart last call review of draft-ietf-alto-cdni-request-routing-alto-16

2021-09-07 Thread Russ Housley
Hi. The vast bulk of the changes resolve my comments. I'm dropping the resolved comments from my reply. > > Section 5.6, 3rd paragraph after bullets: I do not understand the > second MUST statement in this paragraph. The sentence seems to contain > a mix of defining the superset and a MUST

Re: [alto] Opsdir last call review of draft-ietf-alto-unified-props-new-18

2021-09-07 Thread Jensen Zhang
Hi Scott, Thanks for your review. About your questions, please see my answers inline. Best regards, Jensen On Wed, Aug 25, 2021 at 10:08 PM Scott Bradner via Datatracker < nore...@ietf.org> wrote: > Reviewer: Scott Bradner > Review result: Ready > > This is an OPD-DIR review of ALTO

Re: [alto] Artart last call review of draft-ietf-alto-cdni-request-routing-alto-16

2021-09-07 Thread Jensen Zhang
Hi Thomas, Thanks for the very nice review! Please see our responses inline. Thanks, Jensen On Sat, Aug 28, 2021 at 8:28 AM Thomas Fossati via Datatracker < nore...@ietf.org> wrote: > Reviewer: Thomas Fossati > Review result: Ready with Nits > > # Abstract > > I suggest to slightly increase