BUG-2689: Cast exception when serializing L3Address from groupbasedpolicy policy.yang

When deserializing L3AddressKey, alphabetical order of key arguments
didn't match with order of related QNames as defined in yang model.

The binding-independent world retains the ordering specified in model,
while the binding-aware world requires constructor arguments to be
alpha-sorted.

Change-Id: Icb277fed97d0984ca1a60aa7add7817db727facc
Signed-off-by: Martin Ciglan <mciglan@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Updated Json Stream Writer to use gson JsonWriter."

Merge "Fix supporting-node in topology model so it can actually be used."

Fix supporting-node in topology model so it can actually be used.

supporting-node in the topology model *only* has a node-ref
The node-ref is a leafref, which only provides the node-id
for the node.  Since we have no notion of what topology the supporting node
is in, we can't find it.

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

Bug 2361: Added meta-level SPI for building any statement-based models.

This meta-level SPI is designed for components, which introduces
support for conrete statements, identifier namespaces or effective
model inference to the YANG parser.

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

Bug 2497: Added YIN Export for Effective Schema Context.

Added initial implementation of YIN schema export,
which uses current Schema Context and effective API
to infer model.

Once the parser will be updated to preserve declared model,
this implementation will be hugely simplified only to subset
of current implemention - which is implementation
of StatementTextWriter interface.

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

Merge "Bug 2444: Added concrete API definitions for YANG statements."

Fix maven plugin not recognizing transitive interfaces

An implenter does not need to directly implement the interface, but can
rely on it being captured somewhere else. Instead of custom magic, use
Class.isAssignableFrom() to check for compatibility.

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

Bug 2444: Added concrete API definitions for YANG statements.

Introduced type captures of DeclaredStatement for RFC6020 statments
along with some convenience getters to decrease need of users
to look up statements in collections.

This getters conveys RFC6020 semantic, but does not prevent
access to any statements which may be introduced in later
versions of YANG.

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

BUG-2179: split logging out of CodeGenerator

Currently SPI requires downstream users to import maven infrastructure
even if they are not using it. Split the two maven-related injection
methods into dedicated interfaces.

MavenLogAware is automatically deprecated, as maven 3.1+ works with
slf4j out of the box, hence Log injection is no longer needed.

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

BUG-2179: remove plexus-slf4j-logging

Maven 3.1.1 already supports slf4j as its native logging API, no need to
pull in the bridge.

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

Merge "Update Objects->MoreObjects"

Merge "Do not override inherited groupId"

Updated Json Stream Writer to use gson JsonWriter.

Custom implementation of JSON writing methods has been replaced with native
com.google.gson.stream.JsonWriter methods. Among other things, this rework
resolves issues (bugs) with escape characters in string values being written
into Json (e.g. new lines in strings etc.).

Important notice: com.google.gson.stream.JsonWriter enforces that output
must start with '{' and end with '}', otherwise
'java.lang.IllegalStateException:Nesting problem' is thrown.

So our original output looks for an example like:
"my-module:top-container": {
...
}

Now, new output looks as following:
{
  "my-module:top-container": {
  ...
  }
}
.

Change-Id: I8073285d003015721d54887ddbff0484d7db89c8
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Merge "Bug 2512: Added skeleton of Statement Source APIs."

Update Objects->MoreObjects

Guava 18 introduces MoreObjects so it does not clash on naming with
Java7 or later. Also migrate Objects.equal() to Objects.equals().

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

Do not override inherited groupId

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

Merge "Remove features-builder"

Merge "Add a warning about features-test being moved"

Merge "Remove xtend.version definition"

Bug 2512: Added skeleton of Statement Source APIs.

Introduces StatementStreamSource and StatementWriter,
in order to support any YANG statement-based serialization
of YANG model without requiring concrete statement model.

This allows common APIs for YANG text and YIN XML parsers
in order to produce enough information to infer effective
model.

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

Remove <repositories> and <pluginRepositories> sections

It is recommended that developers and servers configure this locally via
settings.xml.

https://lists.opendaylight.org/pipermail/discuss/2015-January/004482.html

Change-Id: Ifc89c520b593abedaca80be3706775bef747db15
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove features-builder

The functionality has been moved to odlparent, so there's no need to
retain it here.

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

Add a warning about features-test being moved

We have migrated the code away and will be killing this one in Lithium.

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

Remove xtend.version definition

There is no longer a need for it.

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

Fixup build failures introduced by guava 18

This fixes the fallout from the version skew.

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

Merge "Bug 2512: Initial design of YANG statement meta-model."

Bug 2512: Initial design of YANG statement meta-model.

Analysis of several new functional requirements for YANG parser,
showed  need to define meta-model of YANG model, which
could be used for transformation and processing of different YANG model
serialization formats (YANG text and YIN).

This new model introduces additional concept from YANG:

  - Statement
     - DeclaredStatement
     - EffectiveStatement
  - StatementDefinition
  - IdentifierNamespace

This basic building blocks allow us to describe with these APIs
any YANG statement model, which may be introduced down the line
in form of YANG extensions or updates to YANG specification,
which does not affect statement principle.

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

Merge "BUG 2413 - tracking in yang grouping"

Merge "BUG 2307: Filtering proxy for Schema context functionality"

BUG 2307: Filtering proxy for Schema context functionality

Change-Id: If4c2231993ae70144f677625df100fa078695863
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Merge "Migrate to use odlparent features-test"

Merge "Remove repository declaration"

Migrate to use odlparent features-test

The code from features-test has been migrated over to odlparent. This
patch makes sure we use that artifact instead of the yangtools-provided
one.

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

Bug 2624 - XXX.getQName() should return namespace of module to which XXX
belongs.

For more information, please see the Bug 2624.

Change-Id: I47bdc55bb90ad91a31110f9edfa842a39bcdf9ec
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Merge "Make JSON serializer always serialize empty containers"

Make JSON serializer always serialize empty containers

Even if their presence is not required

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

BUG 2413 - tracking in yang grouping

Change-Id: I284006a735c711ff9faf4d9e953906bb8f21fa06
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Remove repository declaration

Repositories are managed by settings.xml, so remove them from parent.

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

Mark yang generated source location as a source directory.

Change-Id: I958affa226e9e34a795ffea42450976ca9ef4697
Signed-off-by: Nathan Harmon <nathan.harmon@hp.com>

Make dependencyManagement mandatory

Activating on existence of src/main/yang leads to different things being
defined when this pom is used in build vs. offlne.

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

Removed unused dependency on yang-binding from yang-model-util

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

Add notification support to SchemaTracker

This allows serialization of notification from NormalizedNode writer

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

Fix possible null pointer in NormalizedNode XML stream writer

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

Merge "Introduce javax.annonations to concepts"

Merge "Document the Path concept."

Merge changes I7aea2237,Ie8e125e5

* changes:
  Remove ietf-netconf
  Eliminate unused pluginManagement section

Merge changes Id14ebb05,I2a431693

* changes:
  Add copyright header and fix whitespace
  Fix lifecycle mapping errors in eclipse

Merge "Optimize Types instantiation"

Merge "Improve codec performance"

Merge "Modified RpcResultBuilder classes to be Serializable"

Introduce javax.annonations to concepts

Annotating expected behaviours guides analysis tools, so it is useful to
use them in concepts.

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

Document the Path concept.

Adds a javadoc clarification to the concept, which allows its
implementations to be documented as well.

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

Modified RpcResultBuilder classes to be Serializable

Data store exceptions (eg TransactionCommitFailedException) can be
thrown across the wire by remote actors in the clustered store and
they contain RpcError instances so RpcErrorImpl must be Serializable.
Also RpcResultImpl should be Serializable.

Change-Id: Ia938d0fd5f51400ab0c7ae084ef16f22a99de24e
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Bug 2561: Parser incorrectly allows usage of union in list"

Resolved some sonar issues: Dead store to local variable

DataValidationException - local variables actualValue and expectedValue in method checkListKey
were used only as parameters by throwing IllegalListKeyException, which was thrown when childNode was null.

This check was removed because:

1. it is already performed in call to overloaded checkListKey
2. value is already dereferenced and if it were null a
NullPointerException would have occurred

JSONNormalizedNodeStreamWriter - removed waste calling of anyxmlNode method

BindingToRestRPC - in method invoke added error, which occurs during extracting rpc
output in proxy method, to the rpc errors.

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

Remove ietf-netconf

We never built it, so remove it completely from repository.

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

Add copyright header and fix whitespace

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

Eliminate unused pluginManagement section

This is an aggregator, no need to define further plugins.

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

Fix lifecycle mapping errors in eclipse

maven-checkstyle-plugin needs to be added to lifecycle mapping. Also do
not override the version provided by odlparent.

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

Improve codec performance

Convert the codecs from reflection to invocation API. Invoking
constructors and getters via reflection seems to be causing around
2% overhead which can be avoided if the call site is properly bound.

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

Optimize Types instantiation

Instead of using Arrays.copyOf(), use a simple array clone -- it has
better performance characteristics.

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

Merge "Bug 865: ModuleInfoBackedContext is no more SchemaSourceProvider."

Merge "Bug 865: Moved filesystem model cache related functionality"

BUG-1434: fix NPEs when using null byte arrays

Commit introduced a regression, which caused NPEs being thrown when the
byte array was null (and hence failed to copy). Guard the copy with an
explicit null check and do not use Arrays.copyOf(), but clone().

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

Bug 865: ModuleInfoBackedContext is no more SchemaSourceProvider.

ModuleInfoBackedContext is only used as only class-loader registry
for generated classes and and utility class to create Schema Context
from Binding v1 jar artefacts.

Thus it is safely remove implementation of unused and now deprecated
interface SchemaSourceProvider.

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

Bug 865: Moved filesystem model cache related functionality

Moved static utilities from deprecated implementation of file-system
cache to non-deprecated Helium implementation in order
to ease removal of deprecation classes.

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

Merge "Cleanup warning suppressions"

Merge "Remove unused imports"

Merge "Migrate ClassLoaderUtils callers"

Merge "Cleanup warnings in pom.xml"

Merge "Bug 1434 - Make arrays non-mutable"

Bug 1093: Extracted major static inner classes in parser.builder.impl

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

Cleanup warning suppressions

Some of these are implied by context, some are needed down the stack.

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

Remove unused imports

The usual spring cleaning of unused import warnings.

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

Migrate ClassLoaderUtils callers

oeg.opendaylight.yangtools.yang.util.ClassLoaderUtils is deprecated in
favor of the util package. This switches over some users.

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

Cleanup warnings in pom.xml

- do not override managed versions
- do not override parent groupId where not needed
- do not override parent version where not needed

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

Merge "Split out codecs from BindingCodecContext"

Merge "DataTreeCandidateNode.getChildNodes() should return a Collection"

Merge "Make NodeModification return a Collection"

Merge "Fixed incorrect serialization of multiple nested UnionTypes."

Bug 2561: Parser incorrectly allows usage of union in list

Proposed solution rejects this and parser throws exception

Change-Id: I3cca06a72cfa33245f05a81e62b6a696966697c4
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

Bug 1434 - Make arrays non-mutable

- changed getters, if fields return type is array,
  create local copy and send temporary reference
- constructor which has field of type array,
  save this array by temporary reference

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

Merge "Bug 2480: Union objects are generated incorrectly when using bits type"

Split out codecs from BindingCodecContext

Static inner classes reduce code clarity, so split them out before we
start making changes.

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

Use Maven Enforcer plugin to require Maven version

Using the <prerequisites> configuration in pom files is deprecated and
it is recommended to use the maven-enforcer-plugin to enforce these
rules instead.

See:
http://jira.codehaus.org/browse/MNG-5297
http://jira.codehaus.org/browse/MNG-4840

Change-Id: Ia0265ba264693c3ce09ebb24a6a489bad5efa60e
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

DataTreeCandidateNode.getChildNodes() should return a Collection

All known implementations actually use a collection and users usually
need to know the size to size their structures appropriately. It is fair
ask to specify the return to be a Collection, with undefined iteration
order.

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

Make NodeModification return a Collection

Knowing the number of nodes is often useful. Our internal implementation
uses a Map, so we can freely promote this to a Collection.

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

Merge "Added getModifiedChild to DataTreeCandidateNode."

Merge "Bug 2404: RPC and Notification support for Binding Data Codec"

Merge "Bug 1305: Review YangTools grammar"

Fixed incorrect serialization of multiple nested UnionTypes.

Change-Id: I2bf4153fc3b931fd6f80f2683b8abb47a1e12c7b
Signed-off-by: Tomas Cere <tcere@cisco.com>

Bug 1305: Review YangTools grammar

Since our grammar uses strings, which are validated in Java code during entering actual statements,
this has been addressed within this commit. Rest of grammar issues have been opened as bugs in bugzilla.

Change-Id: Ia70afeeaa1b59d1b77076b1c71cdacf8c3779d74
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

Bug 865: Deprecated Hydrogen Source Repository APIs

Deprecated Hydrogen Source Repository APIs, which are not
used in Helium MD-SAL runtime code and
migrated necessary utility methods from deprecated classes
to non-deprecated ones.

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

Merge "Improve BaseTemplate speed"

Merge "Bug 2431: YangTools grammar misses unknown statement in some statements"

Merge "Added missing copyright headers to yang-data-impl"

Added getModifiedChild to DataTreeCandidateNode.

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

Added missing copyright headers to yang-data-impl

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

BUG-2606: add support for removing augmentations

This patch adds an explicit removeAugmentation() method to the builder.

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