BUG-997 Add SourceIdentifier to MissingSchemaSourceException

Change-Id: Ia4a595492935a66371faa844c5286a312a7897e2
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-997 Fix unused argument in SchemaResolutionException

Change-Id: Id40a5318d824e71405a220c18dffa34ade745c08
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-997 Implement Filesystem source cache for schema repository

Use utility methods for storing files from legacy Filesystem cache

+ Add sorting by Cost to schema repository for multiple providers of the same source

Change-Id: I9a30240042e34fe7898966467b503eee85166887
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-997 Add tests for shared schema context and fix dependency resolution bug

Change-Id: I51d1110629b915e6893a08fa827ccf2ba46ed181
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "BUG-1436: deprecate IntermediateMapping"

Merge "Fixed incorrect test location."

Merge "Fix AugmentationSchemaProxy.getDataChildByName()"

Migrate test asserts

With update to JUnit 4.11 we need to migrate tests to use
proper non-deprecated methods/classes. Also removes wildcard imports.

Change-Id: I969f482a2dd85242fd32c22e7c5807d643e3e554
Signed-off-by: Robert Varga <rovarga@cisco.com>

Do not override versions provided through odlparent

Cuts down declarations which are duplicates and/or result in downgrades.

Change-Id: I7110996ddedff456881a768cd49841191e219345
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix AugmentationSchemaProxy.getDataChildByName()

This implementation breaks the API contract specified by
AugmentationDefinition by throwing IllegalArgumentException where it
should just return null.

Drive-by: fix a typo in AugmentationSchema
Change-Id: I762ce3c02ba776d8a025ccd5bcdba82a6c1579ad
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1436: deprecate IntermediateMapping

IntermediateMapping is only used in CompositeNode-related codecs and
thus is scheduled to be removed at the same time as CompositeNodes.

Change-Id: Ifff06e62441e5f8cc2eac069145389201f369493
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Bug 1372 - toString methods in generated classes"

Merge "BUG-576: modified parser to handle augmentation of extension instances."

Merge "BUG-576: modified parser to handle refinement of extension instances."

Merge "Fix sonar not reporting unit tests"

Fix sonar not reporting unit tests

Enables jacoco plugin (uncodnitionally for now) to get sonar to pickup
the stats.
Changed TypeDefinitionCompilationTest because synthetic fields.

Change-Id: I6baced1cd3ed1a10b790c8a7f11348febc1a380b
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Ladislav Borak <lborak@cisco.com>

BUG-1487: refactor StreamWriterGenerator

This is a reorganization of the StreamWriterGenerator such that it is
properly split up into classes, with contracts clarified. It also
prevents outside subclassing. Also cache pre-made serializer instances.

Change-Id: I492d73a082c24c03b8d17f3868e4a497c8f64b7b
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1487: use prototypes for instantiating codecs

Optimizes the per-class overhead by copying a prototype class, and then
setting the proper serializer name. At the same time we get rid of the
static serializer methods, making the interface a bit cleaner.
Additionally, we hide the AbstractStreamWriterGenerator, so that to have
a proper boundary.

Change-Id: Ia7d56ff16e1eec7204fa087c49f2ee55a658a377
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-576: modified parser to handle augmentation of extension instances.

Change-Id: Ic40d5691baa57ee72953abf761dd52e1c85f66c2
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Bug 1432 - Generated DTOs for types use inefficient instantiation"

BUG-576: modified parser to handle refinement of extension instances.

If user tries to refine an instance of extension parser log this event and continue with parsing
instead of throwing an exception.

Change-Id: I6d535b124527bc5a47ef90ba72b57eccbd6c73ef
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Bug 1432 - Generated DTOs for types use inefficient instantiation

- changed method for generating default instance value
- only for types (boolean, byte, short, int and long)

Change-Id: I4a423f3569ed84f4d6aff8efe97a110045c6e7c7
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Fix feature file (again :( )

Change-Id: I95c9d241a812f6d54b7df6d5ee46627930bc1691
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Fix to feature file

Change-Id: I6619218e29d8581aedb8012e3fd35553168ad36d
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Merge "yang-data-json is in the feature file, but not built"

BUG-1402: Make sure the version in the yang-maven-plugin-it is interpolated also for the parent

- The parent version of the test parent pom for the IT was still
hardcoded, we need that to come from the project itself else it cannot
be released

Change-Id: I88ae85df3bb84e805df9594b56d3c62d21386f4f
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

yang-data-json is in the feature file, but not built

- Remove reference to a no-longer existent artifacts and add new dependencies

Change-Id: Ib9fe2e388f658114e0deadfa4dbab0f403a1fed7
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

BUG-1382: eliminate use of QName.getPrefix from yang parser

Updated tests - removed tests on QName's prefix.

Please do not merge before https://git.opendaylight.org/gerrit/#/c/9640/ is merged.

Change-Id: I41b8940965631609074c5bb8e4c2e0538a33f6a9
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "BUG-1472: deprecated CompositeNode and friends"

Merge "BUG-994: introduce ObjectCache for QNameModule/QName"

BUG-1472: deprecated CompositeNode and friends

These APIs are deprecated in favor of NormalizedNode and friends. Mark
them as such, so the code that needs to be migrated gets highlighted.

Change-Id: I9d5b903d54dd295f4111f6832bf023b2685c0291
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 1439, Bug 1443 Binding Codec NormalizedNodeWriter support

Added initial implementation of Code generator for Binding Stream Writers
which uses NormalizedNodeStreamWriter on the backend
to write data.

Change-Id: Icae22fd1b9ed4256022650d3ad16b38c577f3d3a
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-1275: teach NormalizedNode builders about size hints"

Merge "BUG-865: Remove SchemaServiceListener"

Merge "BUG-1382: deprecate QName.getPrefix()"

Bug 1232 - Add snippet of yang file into comments

- reworked all proposals from Tom Pantelis
- tested with toaster.yang, generated javadoc files added
  as attachment into bugzilla

Change-Id: I6a6c8a6f6592b1abf9368e9b8232263a029460a8
Signed-off-by: Ladislav Borak <lborak@cisco.com>

BUG-1382: deprecate QName.getPrefix()

Prefix storage in QNames is deprecated. Mark the constructors and
getter as deprecated so it can be removed at a later stage.

Change-Id: I68637fe156195b3668cce334b1cc70ab322c0ffc
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 1424: Documented RootModificationApplyOperation

Change-Id: Ided408fb2aeff572eddbc226a8b97c288760b5a3
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 1475: Correctly handle case of typedef that extends boolean"

BUG-994: introduce ObjectCache for QNameModule/QName

This introduces caching for QNameModule using object-cache-api. This
will allow for efficient object reuse, at the cost of requiring a cache
implementation being available on the class path.

The use of cache has been not propagated to the static creator methods,
simply because the caching lookup introduces a cost which is not
acceptable in all contexts.

Change-Id: I5af9c7e54c726c05a5824521f5ed5183dc4899b4
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1275: teach NormalizedNode builders about size hints

With the upcoming codec work we will be able to offer sizing hints
when instantiating normalized nodes. This patch exposes the methods to
pass those hints down to the implementation maps, so they can size
themselves.

Change-Id: Idbc39bda3937d11c89b89807bd253afca5d775f7
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove unused imports

Lowers the number of warnings a bit.

Change-Id: I9cbb9f0bd2f711fa4f7410a5023c73497a59f35e
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-865: Remove SchemaServiceListener

Eradicates the long-deprecated SchemaServiceListener.
SchemaContextListener is the new name for the equivalent service.

Requires controller to merge I15d495d805eb7a08c3bcd3ab0cf8b5a8d2797db5.

Change-Id: I38185b131e9745c0cad366c38428e8be9a8fb4fe
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 1475: Correctly handle case of typedef that extends boolean

Change-Id: I892ddb39b56cbd17ea22f07a94f4c2c2dd9ed448
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "BUG-1275: Expose XmlStreamUtils.writeValue()"

Merge "BUG-997: Evolve the SchemaRegistry concepts"

Merge "BUG-1469: introduce ReflectiveExceptionMapper"

Merge "BUG-1473: rename yang.model.util.InstanceIdentifier"

Merge "BUG-1460: make sure DCL works"

BUG-1473: rename yang.model.util.InstanceIdentifier

The class is renamed to InstanceIdentifierType and users are updated to
point to the new name.

Change-Id: I7ece7ea97a4bbbc623156e0e9db692d1e2a56f81
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-997: Evolve the SchemaRegistry concepts

This is the next step in SchemaRegistry evolution. Introduces
SchemaSourceListener and abstract classes for cache and transformer
patterns. Also completes implementation of the the abstract schema
registry and converts URLSchemaContextResolver. Finally it marks the
APIs as Beta, since they are being proven until we certify NETCONF
completely.

Change-Id: If82522c9a9a1913c818146da1247c8c5ca79b9c5
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1469: introduce ReflectiveExceptionMapper

Convenience class for use when simple instantiation is acceptable and
the mapper instance can be shared.

Change-Id: I1946155a46b94fda5067ccbffcbf3b72f2ce8e3d
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Updated creation of Binding Spec Context to provide additional information."

Merge "Runtime Codec cleanup"

Updated creation of Binding Spec Context to provide additional information.

Change-Id: I0c71d8b3cb3a569b340fc0ef0c71283382fe2ac6
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Runtime Codec cleanup

- Extracted Augmentation getter to Bidning Reflections
- Fixed retrieval of IdenfierCodec

Change-Id: Ib6b6f45349bd8b5b055dccd036549ec727c0186d
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 1437, Bug 1438 NormalizedNode Stream Writer API and Implementation"

Merge "BUG-997: make sure we carry YANG text around"

Bug 1437, Bug 1438 NormalizedNode Stream Writer API and Implementation

Initial draft of NormalizedNode Stream Writer API, along
with initial implementation which produces immutable NormalizedNodes.

Change-Id: I45513d88eefca7aa070e6679ccfdedf358266d59
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-997: make sure we carry YANG text around

Unfortunately we need the schema source for module builder, in order to
support netconf monitoring. This is not right, as that component should
be talking to the schema repository to get the text version of a
particular module.

Change-Id: I3360c5b060f63b415f6248ae6e05d41136247ed0
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Bug 1119 - Optimize generated range checks"

Bug 1119 - Optimize generated range checks

- moved initialization of range and length constraints list
  into static initialization block

Change-Id: I2f5631fceb0e776fe3172e9a05ee97b7fe81252b
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "BUG-1305: fixed unknown nodes and issue with single quote in antlr grammar."

Merge "Bug 1436: Introduced BindingStreamEventWriter interface"

Merge "Bug 1430: New common/util concurrent classes"

Bug 1436: Introduced BindingStreamEventWriter interface

Introduced BindingStreamEventWriter interface,
which allows to implement Binding codecs
with plugable backend for actual serialization.

This interface needs to present in yang.binding
package, since this in only package which
is imported by all generated YANG Java Bindings
and thus visible for them.

Change-Id: I2d235cd63ecda1d63c8077854a38a5dc9d184d2d
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-1305: fixed unknown nodes and issue with single quote in antlr grammar.

Change-Id: I6721cfc3b2042f3564996f08d155201cc2ff2d43
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "BUG-997: introduce base SchemaRepository"

Merge "BUG-997: Rework URLSchemaContextResolver"

Merge "BUG-997: Introduce ASTSchemaSource"

BUG-1460: make sure DCL works

As the bug correctly notes, the check of non-volatile field can result
in bad effects.

Change-Id: Ifd0d72c28fe53692b037c2ecf861d36d312b4182
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-1402: removed hardcoded plugin version for testing."

Merge changes Ic9936326,I9b600761

* changes:
  BUG-865: move ClassLoaderUtils into util package
  BUG-865: prepare to move ListenerRegistry

Merge "BUG-865: Move Immutables"

BUG-576: fixed bug in parsing of union types.

Parser creates an empty collection of restrictions and tries to set them to union builder, but union cannot be restricted.
This patch avoid adding restrictions to union builder.
Added test.

Change-Id: Ie0822f21380f8e5f9cd42960e7248927710053cd
Signed-off-by: Martin Vitez <mvitez@cisco.com>

BUG-997: introduce base SchemaRepository

Change-Id: I51d6c14a217b008afd0fa73c18c6dfc6cc8909b7
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-997: Rework URLSchemaContextResolver

This patch introduces an alternative implementation of
URLSchemaContextResolver, which takes advantage of the newly-created
APIs. The core improvement lies in the fact that it pre-validates
registered sources, thus eliding multiple parse actions.

Change-Id: I3311a3f0251868f91c385d0291365934ccfb9ede
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Introduced CompositeNode <-> NormalizedNode serializer"

Introduced CompositeNode <-> NormalizedNode serializer

- add AnyXmlNodeBase parser and serializer
- in ContainerNodeCnSnSerializer and MapEntryNodeCnSnSerializer was used
MutableCompositeNodeTOImpl which uses init() method which is deprecated
therefore it was rewritten and also whole package was renamed.
- support for AnyXml yang type to yang-data-composite-node

Change-Id: I1f40beef29e8c1c052113043f7029fbc0650192c
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Revert "Composite node <-> normalized node serializer and deserializer""

Revert "Composite node <-> normalized node serializer and deserializer"

This commit renamed artefact, which was missed on code
review done by me, so reverting it for now, to be properly
documented.

Change-Id: Iffaa2f6e1c0e245adbc10ae59cdbc3eb2f365346
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-1402: removed hardcoded plugin version for testing.

In test-parent/pom.xml was hardcoded yang-maven-plugin version in property it-project.version.
This patch remove this property and use parent pom version.

Change-Id: Ib6478c839abcc54a0862439b201b4889f2af5cc7
Signed-off-by: Martin Vitez <mvitez@cisco.com>

BUG-997: Introduce ASTSchemaSource

ASTSchemaSource represents an intermediate processing step, where we
have parsed the YANG text and extracted dependency information.

Change-Id: Ice0b841404a28912787b83f88f9b2d28c68497d5
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Composite node <-> normalized node serializer and deserializer"

BUG-1457: fixed parsing of max-elements statement.

If max-elements was set to "unbounded", parser throws an exception.
Added test.

Change-Id: I37df99b504e989353f5a0b33f0a2d99e6c796668
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Composite node <-> normalized node serializer and deserializer

- add AnyXmlNodeBase parser and serializer
- in ContainerNodeCnSnSerializer and MapEntryNodeCnSnSerializer was used
MutableCompositeNodeTOImpl which uses init() method which is deprecated
therefore it was rewritten and also whole package was renamed.
- support for AnyXml yang type to yang-data-composite-node

Change-Id: Icef3549e402eee0da074900a3db6e2959a5a67a5
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Merge "Bug 1354: Added rpcReplyToDomNodes method."

BUG-865: move ClassLoaderUtils into util package

Removes the two previously-deprecated instances, deprecate the third one
and instantiate the utility at the proper place.

Change-Id: Ic9936326fb0f4d4b2129562ae84b131b79746c23
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-865: Move Immutables

Immutables is a utility class, so it should go into util. Its
capabilities can be expanded there to account for guava classes.

Change-Id: I6f93befd03249cb83a842a53600260b776822ba2
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-865: prepare to move ListenerRegistry

This will evacuate the concepts.util package, guiding users towards
plain util package.

Change-Id: I9b600761f6cd7950b366e2bf40fd85f4bc8d5e9b
Signed-off-by: Robert Varga <rovarga@cisco.com>

Optimize binding types memory usage

Profiling SP edition shows that we have ~30000 empty ArrayList instances
retained by binding.generator.generated.type.builder objects, wasting
around 800kB of memory.

This patch introduces LazyCollections, which allow us to build up these
lists in a manner which uses minimal collections to support the elements
currently present.

Change-Id: I4676356f7684c9cdd152c98a65a240d0c5b94259
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1275: Expose XmlStreamUtils.writeValue()

This exposes a utility method needed for RESTCONF datastore notification
delivery.

Change-Id: Ica14b6a11ecb95da306c37790e6a2fbf5533bfc1
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-1281: fix SchemaContext module ordering"

Merge "BUG-1305: improved antlr grammar handling of unknown nodes."

BUG-1281: fix SchemaContext module ordering

This patch fixes the wildcard lookup and optimizes it by precomputing
the revision.

Change-Id: Icad0a9435aacc4b5401130608833d2cb45e3624f
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1092: rename data.api.InstanceIdentifier to YangInstanceIdentifier

On popular request to clear up confusion between
yang.binding.InstanceIdentifier and yang.data.api.InstanceIdentifier, we
rename the latter (which is the low-level one). This will make it clear
for users of GUIs, who were confused by the tools giving them the choice
between the two.

Change-Id: I1e30a0df93d8bfb891d047c58abcb117d4006c76
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1305: improved antlr grammar handling of unknown nodes.

Removed useless unknown_statement2 and unknown_statement3 definitions

Improved antlr parser to propertly handle extension description by moving unknown_statement definition as last
statement in extension_stmt definition to properly handle description.

Change-Id: I9d8da69ac9336feee07855e85fad5ee2e4033f12
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "BUG-461 Test transformation of rpc input using XmlDocumentUtils"

Merge "BUG-1413: fixed bug in antlr grammar."

Bug 1354: Added rpcReplyToDomNodes method.

Added rpcReplyToDomNodes method into XmlDocumentUtils to convert XML Payload from Rpc Reply message into
Composite Node structures.
Added test and test files (schema, test payloads) for rpcReplyToDomNodes method.

Change-Id: I45e01d58689ad448cdf6e8541cb680db62667d7c
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>