Wen Bo Li

Results 167 comments of Wen Bo Li

> For POLL subscriptions, after each set of updates for individual poll request, a SubscribeResponse message with the sync_response field set to true MUST be generated. It reads to me...

I think this is a good question, so the three candidates I see are 1. server concatenates updates in a JSON list. Concatenation is necessary because the query path could...

Thanks Rob for the clarification, I made a typo there, I agree that a JSON object should be returned whenever the path is a non-wildcard path. Per the [gNMI path...

goyang/ygot's failure is due to an outdated CI image. When you retrigger the CI now it should bypass the current failure.

- Does YANG allow changing a node via deviating and then augmenting it? Does `pyang` pass the check? - Does YANG allow multiple nodes of the same name, but different...

I see, I was putting the `augment` and `deviation` statements in the same YANG file. After I separated them into a different one like you did it worked. It must...

Can you provide an example of where without `deviation` or `augment`, we'd have a conflict? `uses` creates nodes [where it's used](https://tools.ietf.org/html/rfc7950#section-7.13), not where the `grouping` is defined.

Since this is a ygot-specific issue, i.e. merge the modules without accounting for namespace: https://github.com/openconfig/ygot/blob/50b44fcf38c0a20a5828232e058ad58ce1204d02/ygen/codegen.go#L1093-L1105 I think we can fix this particular issue just within ygot, such that as far...

@robshakir Yes I agree `augment` can cause conflicts. My point is that for goyang, the conflicts can only arise from an `augment`. If we add the prefix to the Entry...

Thanks @midakaloom, it looks like this is a bug in the error handling code. I've verified that capturing the error reports the `Duplicate node "c"` error message on Rob's example...