Re: [GROW] WGLC: draft-ietf-grow-simple-va

[Nick Hilliard <[e-mail]>]

nits:

- please run: s/it's/its/s on the document.

- "In some protocol's implementations for example BGP loc-RIB can be
further...".  Change to "For example, in some protocol implementations BGP
loc-RIB can be further..."

- "Let's illustrate...".  Too informal.  "We illustrate..."

- "Another deploymet consideration" => "Another deployment consideration"

- "Therefor under the above intra" => "Therefore under the above intra"

Most of the document looks fine.  However, there is one content omission,
namely that there is no analysis of what happens when a compressed RIB
undergoes a topology change which modifies a bunch of next-hops.  If this
happens, the fib compression can explode with an upper bound of the
uncompressed size.

This may cause Interesting and Unexpected side effects - e.g. routers
falling over due to topology changes and so forth.  From an operational
point of view, I think this is probably important enough to warrant a
mention in the draft.

Nick