Re: [Netconf] comments on draft-ietf-netconf-subscribed-notifications-12

[Kent Watsen <[e-mail]>]

Sure, but can YP import the "Event Record" term from SN?

Also, I think that the definition could be improved.  It currently reads:

   Event record: A set of information detailing an event.


Kent // contributor



> From: Martin Bjorklund, June 8, 2018 10:23 AM
> 
<snip>
> But the name of the leaf doesn't change the semantics.  The description says
> "number of event records", so your casual user still have to understand that a
> YANG Push notif is an event record.
> 
> BTW, it is not clear from the YANG push document that a YANG push notif
> really is an event record.  It uses the term "update record", and use the term
> "event record" in just one place.  This term should be imported from
> Subscribed-notifications, and used.

Update record has additional constraints.  Probably the way to handle this is to tweak the definition of update record in YANG push to the following:

"Update record: An event record which contains a representation of one or more datastore node updates.  In addition, an update record may contain which type of  update led to the datastore node update (e.g., whether the datastore node was added, changed, deleted).  Also included in the  update record may be other metadata, such as a subscription identifier of the subscription as part of which the update record was generated."

Eric

> /martin

_______________________________________________
Netconf mailing list
[e-mail]
https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_netconf&d=DwICAg&c=HAkYuh63rsuhr6Scbfh0UjBXeMK-ndb3voDTXcWzoCI&r=9zkP0xnJUvZGJ9EPoOH7Yhqn2gsBYaGTvjISlaJdcZo&m=ChdyY-tKTlPquVOwbOXya3_PcGgirMepotW7UY8_V68&s=Bd-V3zeqTlVJmJa9OVqUUYf4I2RqZU5ce49Jd2xY37E&e=