[MDSAL-836] Move generated YangModelBindingProvider and ResourceYangModuleInfo Created: 01/Aug/23  Updated: 26/Jan/24  Resolved: 18/Jan/24

Status: Resolved
Project: mdsal
Component/s: Binding codegen
Affects Version/s: None
Fix Version/s: 13.0.0

Type: Improvement Priority: Medium
Reporter: Robert Varga Assignee: Robert Varga
Resolution: Done Votes: 0
Labels: pt
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

We currently generate $YangModuleInfoImpl and $YangModelBindingProvider into org.opendaylight.yang.gen.v1.NAMESPACE package.

The naming of these two classes is rather unfortunate, as it starts with $ due to the requirement not to conflict with other generated classes.

At the end of the day, though, these are separate constructs. Allocate a new top-level package name to host these: org.opendaylight.yang.svc.v1, appended with the namespace, and name these YangModuleInfoImpl and YangModelBindingProviderImpl in that package.

At the end of the day:

  • org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.restconf.subscribed.notifications.rev191117.$YangModelBindingProvider should become org.opendaylight.yang.svc.v1.urn.ietf.params.xml.ns.yang.ietf.restconf.subscribed.notifications.rev191117.YangModelBindingProviderImpl
  • org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.restconf.subscribed.notifications.rev191117.$YangModuleInfoImpl should become org.opendaylight.yang.svc.v1.urn.ietf.params.xml.ns.yang.ietf.restconf.subscribed.notifications.rev191117.YangModuleInfoImplImpl

We already define the leading namespaces in yang.binding.contract.Naming, this needs to be captured there. This also will throw a wrench in assumptions around binding class -> provider/info mapping, so those need to be fixed up, too.


Generated at Wed Feb 07 20:11:06 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.