[YANGTOOLS-1348] Remove Binding-only constructs from YANG parser

Blocks — Tue, 1 Nov 2022 16:41:48 +0000

We have a number of constructs which were introduced during the original PoC evaluation of Java Bindings. Some of these have performance implications on the parser, others serve to confuse users with aspects they should not care about.

There are four distinct parts:

AugmentationSchemaNode.getOriginalDefinition()
DerivableSchemaNode, with its getOriginal()
AddedByUsesAware, with its isAddedByUses()
CopyableNode, with its isAugmenting()

The first two are used exclusively by mdsal-binding runtime components (runtime-api and generator). As demonstrated by ~~MDSAL-694~~, mdsal-binding-generator should have enough structural state to be able to have an internal replacement and provide the right details to BindingRuntimeTypes.

The third is mostly used by the same MD-SAL runtime components, but also two obsolete components (wadl-generator in NETCONF and unified-html-generator in MD-SAL). Parser is giving enough information in DeclaredStatement and Effective}Statement and their layout for mdsal-binding-generator to reconstruct this information for its use and again forward it to BindingRuntimeTypes as needed.

The fourth is more tricky, as it can be reconstructed like the third item, but there are users in NETCONF and RESTCONF, hence those need to be analyzed and a proper course of action be plotted.

OpenDaylight JIRA

[YANGTOOLS-1348] Remove Binding-only constructs from YANG parser