[Bier] Intdir telechat review of draft-ietf-bier-bar-ipa-12

Suresh Krishnan via Datatracker <noreply@ietf.org> Tue, 19 April 2022 03:54 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Suresh Krishnan via Datatracker <noreply@ietf.org>
To: int-dir@ietf.org
Cc: bier@ietf.org, draft-ietf-bier-bar-ipa.all@ietf.org, last-call@ietf.org
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <165034044824.2741.13284922661455065282@ietfa.amsl.com>
Reply-To: Suresh Krishnan <suresh.krishnan@gmail.com>
Date: Mon, 18 Apr 2022 20:54:08 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/bier/mDCNZxisfDt6Gkb9h8eJ8vzUIdg>
Subject: [Bier] Intdir telechat review of draft-ietf-bier-bar-ipa-12

Reviewer: Suresh Krishnan
Review result: Ready with Issues

This document is short and clear and easy to understand for someone who has a
basic understanding of BIER. I do have one minor issue that probably needs to
get addressed/clarified

* Section 3:

Step 3: "If BA is NULL" -> What exactly does this mean? Does this mean BAR==0?
If so, it might be good to state this explicitly and I suggest pulling forward
the text from Section 3.1. to clarify BAR==0 means that BA is NULL and BC is
NULL. If not, can you please clarify? i.e. do you expect there to be a NULL BA
with a BAR<>0?

[Bier] Intdir telechat review of draft-ietf-bier-… Suresh Krishnan via Datatracker
Re: [Bier] Intdir telechat review of draft-ietf-b… Jeffrey (Zhaohui) Zhang