Merge "BUG-576: fixed possible null pointer dereference in BindingGeneratorImpl"

Merge "BUG-1433: augmentation field visibility changed to default"

Merge "Bug 1446: Add new concurrent classes for tracking stats"

Merge "BUG-869: removed redundant check for null before an instanceof"

Merge "BUG-869: added proper handling of nullable parameter"

Merge "Fix to feature files to include ietf-yang-types-20130715"

Fix to feature files to include ietf-yang-types-20130715

ietf-yang-types-20130715 was missing from feature:

odl-yangtools-models

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

Bug 1446: Add new concurrent classes for tracking stats

TrackingLinkedBlockingQueue
    A LinkedBlockingQueue that tracks the largest queue size foridebugging.

CountingRejectedExecutionHandler
    A RejectedExecutionHandler that delegates to a backing RejectedExecutionHandler
    and counts the number of rejected tasks.

DurationStatsTracker
    Class that calculates and tracks time duration statistics.

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

Merge "Bug 1505 - Provide test coverage for binding-data-codec using sal-test-model"

Merge "Hotfix for gson test."

Merge "BUG-869: avoid String repetition in RestRestconfService."

Merge "BUG-576: fixed invalid key used to search map of modules."

Merge "BUG-576: fixed bug in AbstractSchemaRepository.removeSource"

Merge "BUG-576: added support for handling nested groupings in code generator."

Merge "BUG-576: fixed invalid if conditions in LazyGeneratedCodecRegistry."

Hotfix for gson test.

Test had to be modified prior to internal use of hash sets. Test relied on retrieving first item from
parsedData values. That lead to test failure on various platforms. Now test should run without problems.

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

Merge "BUG-576: fixed test sources loading in YangModuleInfoCompilationTest."

Merge "BUG-869: removed useless null check from DataValidationException.checkListKey"

BUG-869: avoid String repetition in RestRestconfService.

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

BUG-869: removed useless null check from DataValidationException.checkListKey

Null check is useless 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

Expression 'if (isLegal == false)' simplified to 'if (!isLegal)'.

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

Merge "BUG-1590: introduce PathArgument.toRelativeString() and use it"

BUG-869: removed redundant check for null before an instanceof

No need to check for null before an instanceof, the instanceof keyword returns false when given a null argument.

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

BUG-576: fixed bug in AbstractSchemaRepository.removeSource

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

BUG-869: replaced inefficient use of keySet with entrySet in XmlUtils

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

BUG-576: fixed invalid if conditions in LazyGeneratedCodecRegistry.

1. fixed check to 'previous instanceof DataNodeContainer' - this check always return true
2. fixed incorrect equals parameter: expected URI but was QName

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

Bug 1505 - Provide test coverage for binding-data-codec using
sal-test-model

* added test yang modules to bindig-test-model
* added tests for serialization/deserialization of normalized nodes and
paths to binding-data-codec

Change-Id: Ibdf24dd9e51281be488fb40481a79d6f086a2115
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

Merge "Fixed incorrect instanceof in yang-data-operations."

BUG-576: fixed test sources loading in YangModuleInfoCompilationTest.

Fixed test to parse File sources with relative test.

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

Merge "Bug 1630: Updated normalized to binding codec to exclude LeafSetEntryNode."

Bug 1630: Updated normalized to binding codec to exclude LeafSetEntryNode.

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

Merge "Fixed merge job breakage."

Fixed merge job breakage.

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

Merge "BUG-869: reworked check for integer value."

Merge "ModifiedNode instantiation improvement."

Merge "Bug 1258: Implement DataTree partial indexing"

BUG-1630: fix warning message format

The bug has been reasolved by change
I4a35ff27725dab7f35b83ae1bcc6d728a313bd1c, this patch just fixes the
message format.

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

Merge "BUG-650: add missing bypass"

Merge "BUG-650: improve toYangInstanceIdentifier() speed"

BUG-650: add missing bypass

ChoiceNode is not representable either, so check for it early. Also
eliminate the FIXME, now that we are in the clear about what needs to
happen. Finally log offending class in warning, so we can track it down.

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

Merge "Pre-cache child node set"

Merge "Bug 1443: Added support for misplaced augmentations."

Fixed incorrect instanceof in yang-data-operations.

Intended check was to check for AugmentationIdentifier,
AugmentationNode was incorrectly checked.

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

Bug 1443: Added support for misplaced augmentations.

Binding Specification reuses elements inside groupings
as part of normal data tree. Augmentations and augmented
cases could be copied by user using copy builders
and may be used at incorrect place, where augmentation
with same data is present.

Binding Data Codec will treat this augmentations as
same, if augmentation definining same data subtree
is present at real location.

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

BUG-650: improve toYangInstanceIdentifier() speed

This removes a superfluous call to getCodecContextNode(), since that
call is already performed by the InstanceIdentifierCodec.

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

BUG-650: clarify BindingNormalizedNodeSerializer API contract

Non-representable YangInstanceIdentifiers are properly reported via a
null return instead of raising an undocumented IllegalArgumentException.

Also does some implementation house-cleaning geared towards performance,
as ArrayList gives better locality.

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

Remove unused arguments from UnionTypeCodec.loader()

The two codecs passed in were unused, so remove them, fix the callsite
and remove the useless warnings suppression.

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

UnionTypeCodec should not throw raw Exception

This is a callable, and we know what types of exceptions can be thrown,
so get rid of the implied sonar warning.

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

Merge "BUG-865: removed use of deprecated YangInstanceIdentifier.getPath method"

Merge "Bug 1505 - Added test model and instance identifier test for augmentations"

BUG-576: added support for handling nested groupings in code generator.

Failing example (causing NullPointerException):

container foo {
  grouping bar {}
  uses bar;
}

Added call to resolve groupings before resolving uses.

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

Pre-cache child node set

Tracing has shown this method gets called ~5.7M times, contributing to
some 10% bootup time. Pre-cache the set of nodes to speed things up.

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

ModifiedNode instantiation improvement.

Every instance of ModifiedNode will contain children with hash map instance based on Modified Node ordering. Now if ModifiedNode
will cary Ordered Children LinkedHashMap will be used otherwise ordinary HashMap will be used. This should improve performance
during validation process.

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

Bug 1258: Implement DataTree partial indexing

Now the TreeNode elements are instantiated lazily per applyWrite/applyMerge/applySubtreeChange operations.
Implemented createTreeNode in TreeNodeFactory to create non recursively single node instance in TreeNode
Renamed create to createTreeNodeRecursively.

Added benchmarks project directly into the root of yangtools project.
Benchmarks project contains benchmark for testing performance of InMemoryDataTree implementation.
Tool used for benchmarks is JHM.

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

Merge "BUG-869: description field in BitsType made static"

Merge "BUG-865: removed UnknownType class"

Merge "BUG-869: avoid unused constructor parameter namespace in YangModelDependencyInfo.ModuleDependencyInfo"

Merge "BUG-869: simplified if expressions as suggested by Sonar."

Merge "BUG-869: fixed code style violations in TypeProviderImpl reported by Sonar"

Merge "Fix for features - added missing dependency"

BUG-1590: introduce PathArgument.toRelativeString() and use it

This patch introduces a new method, toRelativeString(), which does the
same thing as toString(), but takes into account previous path argument
and does not emit namespace if it matches the previous namespace.

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

Bug 1505 - Added test model and instance identifier test for augmentations

Added binding-test-model, which contains models for unit testing
and integration testing of binding runtime components such as binding-data-codec.

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

BUG-865: removed use of deprecated YangInstanceIdentifier.getPath method

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

Fix for features - added missing dependency

There was a missing dependency in the features/pom.xml
In some circumstances, it was causing downstream breakage.
This patch fixes that.

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

BUG-869: reworked check for integer value.

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

BUG-576: fixed possible null pointer dereference in BindingGeneratorImpl

Possible null pointer dereference of resultTOBuilder.

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

BUG-576: fixed invalid key used to search map of modules.

Removed useless null check when searching for grouping.

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

BUG-869: added proper handling of nullable parameter

Argument of Function.apply method is marked as Nullable, but previous implementation assumed it
as nonnull parameter.

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

BUG-869: description field in BitsType made static

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

BUG-869: fixed code style violations in TypeProviderImpl reported by Sonar

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

BUG-866: improved test resource loading

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

BUG-865: removed UnknownType class

UnknownType class is no longer used in yangtools.

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

BUG-869: avoid unused constructor parameter namespace in YangModelDependencyInfo.ModuleDependencyInfo

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

BUG-869: simplified if expressions as suggested by Sonar.

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

Merge ".equals() should check for null"

Improve InstanceIdentifierCodecImpl performance

Instead of key iterator and lookup, use entrySet(), just as suggested by
Sonar.

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

.equals() should check for null

This fixes a sonar warning.

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

Merge "BUG-1589 Exclude xml* prefixes from RandomPrefix"

Allow empty augmentations in Yang modules.

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

Fixed breakage of uses generation in Binding

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

Merge "BUG-576: fixed text file loading in test."

Merge "BUG-1514: fixed bug in module sort."

Merge "BUG-1537: improved YangModuleInfo."

Merge changes If1e9a750,Ib530560c,Id01393c3,I2d375382,Id60447ef,I350611fb,I0dbd80db

* changes:
  BUG-1611: make sure runtime components create short comments
  BUG-1611: force maven plugin generating verbose comments
  BUG-1611: make verbose comments optional
  BUG-1611: use CharMatcher to remove characters
  BUG-1611: drop a FIXME for thread-safety
  BUG-1611: do not instantiate splitters needlessly
  BUG-1611: fix SimpleDateFormat

Merge "BUG-576: fixed grammar to handle union type with only one type definition."

Merge "BUG-576: fixed javadoc warnings."

Merge "BUG-576: if replaced with else-if in enterImport_stmt method."

Merge "BUG-997: Fix URLSchemaContextResolver"

BUG-1589 Exclude xml* prefixes from RandomPrefix

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

BUG-576: fixed grammar to handle union type with only one type definition.

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

Bug 1572: Fixed duplicate and inconsistent allocation of Version

After switching from numeric version objects to non-numeric Version
modifications based on modifications was broken due to duplicate
allocation of versions - one when derived modification
was allocated and other one when modification was commited,
this make all writes effectivelly conflicting, since
same node was seen with two different versions.

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

BUG-576: fixed text file loading in test.

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

Bug 1604: Fixed incorrect check in Normalized Node Writer startOrderedMapNode

Check which was used in startOrderedMapNode was meant to be part
of startMapEntry. Check for startOrderedMapNode should be
same as for startMapEntry, only used builder should be different.

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

BUG-576: fixed javadoc warnings.

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

BUG-576: if replaced with else-if in enterImport_stmt method.

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

BUG-1514: fixed bug in module sort.

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

Merge "BUG 1440 - json stream to normalized node stream writer"

Merge "BUG-1605: Fixup yang-data-api interface"

Merge "BUG-1605: Fix yang-binding interfaces"

Merge "Fix javadocs a bit"

Merge "BUG-997: fix InMemorySchemaSourceCache interface"