Senthil, I hadn't realised you'd published a new version of the draft.
Please CC me if/when you update it again.
I've quickly reviewed the diffs between -06 and -07 :
1. Terminology
" /Any non-IPFIX terminology used to convey NAT events are described in
/ /this section./"
-> which section is "/this/" referring to? Since this paragraph seems
only to serve as an introduction to the third paragraph which only
contains a single exception, would it be better to remove these two
lines and go directly to the third paragraph? :
However, that causes
confusion in terminology used in NAT specific terms and IPFIX IEs.
Any non-IPFIX terminology used to convey NAT events are described in
this section.
2. Introduction
"This document details the IPFIX Information Elements(IEs)"
-> No need to repeat "(IEs)" since this was already explained in the
Terminology section.
-> Remove the duplicated text:
The IPFIX Protocol [RFC7011] defines a generic push mechanism for
exporting information and events. The IPFIX Information Model
[IPFIX-IANA] defines a set of standard IEs which can be carried by
the IPFIX protocol.This document details the IPFIX Information Elements(IEs) that MUST be
logged by a NAT device that supports NAT logging using IPFIX. This document details the IPFIX Information
Elements(IEs) that MUST be logged by a NAT device that supports NAT
logging using IPFIX, and all the optional fields. The fields
specified in this document are gleaned from [RFC4787] and [RFC5382].
5.4. Quota exceeded Event types
-> In the "/The events that can be reported are .../", I'd like to see
the text be identical to the items listed in table 3 to remove any
possible ambiguity.
5.6.8.4.Global Address mapping high threshold reached
-> Extra whitespace at the period: "paired address pooling behavior ."
8.1. New Information Elements / natLimitEvent, natThresholdEvent,
-> typo: " describer in Table below."
-> you should remove the "Table 22" and "Table 23" descriptions under
those tables, because these won't make any sense when the text is
transcribed into IANA's registry. E
8.2. Modified Information Elements / natEvent
-> Again, you can't modify the definitions of the existing values.
P.