Merge "BUG-584: moved yangtools-it under integration-test and renamed to bundle-test"

Merge "Improved sorting of augmentations before code generation."

Merge "BUG 1130 STRING_DEFAULT_CODEC - deserialize can return null"

Merge "Bug 484: Fix XmlDocumentUtils.toDocument to handle anyxml"

Merge "Checkstyle coding-style rules"

Checkstyle coding-style rules

-avoid using tabs
-avoid trailing whitespaces
-no windows eol chars
-indentation offset 4
-imports order - static imports first, alphabetical order
-avoid unused imports

Change-Id: I4ea8193089f7c5cf77ee432c85cb1c8249043495
Signed-off-by: Milos Fabian <milfabia@cisco.com>

BUG 1130 STRING_DEFAULT_CODEC - deserialize can return null

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

Merge "Bug 724: Handle hashCode/equals properly in InstanceIdentifier"

Merge "Bug 1123 - Fixed incorrect augment target resolving"

Bug 724: Handle hashCode/equals properly in InstanceIdentifier

Modified InstanceIdentifier.NodeIdentifierWithPredicates and
InstanceIdentifier.NodeWithValue classes to perform deep hashCode/equals
to handle byte] values.

Added unit tests.

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

Bug 1123 - Fixed incorrect augment target resolving

- Introduced new ParserUtils#findSchemaNodeInModule
  in parser utils
  which uses SchemaPath, Iterable and full QName match
  to lookup builders for augmentations

- Updated augmentation resolution to use findSchemaNodeInModule
  to lookup correct augmentation target using
  full QName match (namespace, revision, localName) instead
  original algorithm using localName only.

- Updated incorrect negative tests for augmentations
  Added conflicting augmentations.

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

BUG-1120: add ExecutorServiceUtil

ExecutorServiceUtil encapsulates the utilities needed to implement a
blocking-queue based, growing thread pool executor service. This is
useful in both the controller and the openflow plugin.

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

Fix a documentation typo

Fixes missing } in deprecation link.

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

Merge "Bug 1027: Fixed generation of static QNAME field"

Bug 1027: Fixed generation of static QNAME field

All YANG modeled entities such as containers, lists
which have Binding-aware interface associated with
it should have static field QNAME with their respective
from YANG models. Only exception are augmentations
which do not have explicit name.

Instance Identifier codec relies on this existence
of QName which was not true for classes derived from
RPC Input and RPC Output

The code was changed to fix this issue:

BindingGeneratorImpl:
  Updated to include QName for all YANG modeled
  data container.

BindingReflections:
  ClassToQNameLoader was updated to derive correct
  qname for RPC inputs and RPC outputs for models
  which was compiled with missing QName field.

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

BUG-584: moved yangtools-it under integration-test and renamed to bundle-test

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

BUG-1120: expose AbstractRegistration state

This patch allows subclasses to infer whether the registration has been
closed.  At the same time we optimize the memory overhead associated with
each instance by not using an AtomicBoolean, but rather a componation of
a volatile int and an AtomicIntegerFieldUpdater.

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

Merge "Bug 1113 - ietf-restconf needs to specify the version of ietf-yangtypes that must be imported."

Bug 1113 - ietf-restconf needs to specify the version of ietf-yangtypes that must be imported.

Modified the internal ietf-restconf to specify the required version of ietf-yangtypes so runtime compilation runs without an issue.

Change-Id: I06f547aee6d26b7865ea93bbcdd76808d46d8151
Signed-off-by: Devin Avery <devin.avery@brocade.com>

API Clarity: Documented YANG Common.

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

API Claratity: Documented package o.o.y.yang.model.util

Documented package o.o.y.yang.model.util,
documented differences between base YANG types
which resulted in different instantiation pattern for
some of built-in types.

Deprecated utility YangTypesConverter which name was
confusing, added replacement methods in BaseTypes
which name is less confusing and more clearly
comunicates its functions.

Improved hashCode and equals after introducing
static constants.

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

API Claritity: Documented org.opendaylight.yangtools.yang.model.util.repo

Improved documentation of poorly-documented public APIs in
org.opendaylight.yang.model.util.repo which are actually used by
existing implementations.

Added package-info which describes abstract concepts for this package
which are not materialized in interface or class in this package.

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

BUG-648: do not keep HashMap$Values around

Profiling has identified that we keep a lot (>600K) of HashMap$Values
objects around. This is caused by our class hierarchy, which forces the
Container nodes to supply their Iterable values at construction time --
which means they have to capture children.values() and wrap it in an
UnmodifiableIterable.

This patch creates splits the functionality of
AbstractImmutableNormalizedNode into two:

AbstractImmutableNormalizedNode, which just holds the identifier plus
the infrastructure layout. AbstractImmutableNormalizedValueNode, which
additionally holds the value.

Container nodes are then made subclasses of the former, which allows
them to provide getValue() dynamically, thus avoiding the unnecessary
instantiation.

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

Merge "BUG-648: use MapAdaptor"

Merge "BUG-648: fix util building"

Improved sorting of augmentations before code generation.

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

Merge "Bug 731: Moved list initialization in UnionTypeBuilder before constructing UnionType."

BUG-648: use MapAdaptor

This converts the un-ordered nodes to using the MapAdaptor utility,
which should result in reductions to the amount of copies being made.

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

BUG-648: fix util building

It turns out that the util package had incorrect build instructions,
resulting in unresolved dependencies. Fix that.

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

Merge "Fixed setting yang sources to modules."

BUG-648: Create MapAdaptor and friends

This introduces the MapAdaptor class, which allows transforming one Map
implementation to another, balancing the isolation and access patterns.

The core of the idea is that we are using patterns which would benefit
from persistent structures where we can take a point-in-time stable
snapshot, update it and publish the new version all the while clients
accessing the old version suspect nothing. So far we have implemented
the isolation by copying maps around -- which is fine as long as there
are not too many elements.

We do have a persistent strucutre (Ctrie), which allows snapshot and
isolation to happen in O(1), but exacts heavy price on access due to
volatile variables.

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

Fixed setting yang sources to modules.

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

Merge "BUG-864: decrease log level when sorting and parsing modules"

Merge "BUG-973: fixed (de)serialization of union type."

Bug 731: Moved list initialization in UnionTypeBuilder before constructing UnionType.

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

Bug 484: Fix XmlDocumentUtils.toDocument to handle anyxml

For anyxml data, a ClassCastException occurs at
XmlDocumentUtils.createXmlRootElement(XmlDocumentUtils.java:180) when
blindly casting the schema instance to DataNodeContainer - the actual
instance is an AnyXmlSchemaNode. To prevent this, the code should first
check "schema instanceof DataNodeContainer". 'childSchema' will be null
but that's expected for anyxml as it doesn't have a schema.

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

Merge "BUG-1070: split off YangSourceContext"

Merge "BUG-981: remove deprecated elements"

BUG-981: remove deprecated elements

This patch removes deprecated constructor and setters, such that the
RuntimeGeneratedMappingServiceImpl can be instantiated in one go.

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

BUG-1070: split off YangSourceContext

This patch just splits up the various static classes out of
YangSourceContext, making the structure more navigable.

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

BUG-994: convert users of SchemaPath constructor

This patch migrates all in-tree users of the deprecated constructor to
the create family of methods.

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

Bug 1076: Updated SchemaPath hashcode to behave as previously.

Change in hashCode of SchemaPath uncovered other issue
where code was expecting sorted set from HashSet.

Updated hashCode to behave as previously defined.

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

Merge "BUG-582: use shared thread-safe SimpleDateFormat"

Merge "BUG-1070: introduce YangSyntaxErrorException"

BUG-582: use shared thread-safe SimpleDateFormat

This concentrates the per-parser DateTimeFormat into the pre-existing
yang-common one, lowering resource parser resource usage, while
remaining thread-safe.

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

BUG-1070: introduce YangSyntaxErrorException

This introduces a proper checked exception, which is trown when a
syntactic error is encountered during parsing of the YANG module text.

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

Merge "BUG-994: introduce SchemaPath.create() and friends"

Merge changes Ib8372167,I1b7aeca9

* changes:
  BUG-382: optimize BooleanType
  BUG-582: optimize BinaryType

Merge "BUG-868: stop using getChildren()"

BUG-864: decrease log level when sorting and parsing modules

Decrease log level from debug to trace.

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

BUG-973: fixed (de)serialization of union type.

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

BUG-868: stop using getChildren()

This removes users of getChildren() in favor of getValue().

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

BUG-382: optimize BooleanType

This optimizes the BooleanType definition, reducing it's class memory
footprint.

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

BUG-582: optimize BinaryType

This optimizes the defintion of BinaryType, such that it is smaller and
faster.

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

BUG-868: Eliminate StringBufferInputStream

StringBufferInputStream has been deprected since JDK 1.1 -- so let's not
use it.

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

BUG-582: Use Splitter instead of String.split()

String.split() instantiates a pattern, which in itself is not efficient.
Splitters are a lot more efficient, so let's use one in this codepath.

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

BUG-582: Optimize string modification

This code could potentially end up with a lot of copying around. Pay a
single copy operation, iterate over the character array once and replace
characters as needed.

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

BUG-582: Optimize instantiated Splitter

This just turns the string-based splitter into a char-based one, saving
both memory and CPU cycles.

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

BUG-582: reuse SimpleDataTimeUtil

Thisreuses the already-existing thread-local safe formatter, squeezing
extra performance in this path.

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

BUG-582: Optimize and correct validateJavaPackage

This removes obsolete set and uses it from Binding. Also does the same
for the package prefix -- this in fact was incorrect a happened to work
because YANG version == 1, same as our binding version. Finally we
optimize the validateJavaPackage() since it is a major CPU hog.

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

BUG-582: Operate on characters when checking for leading digit

Do not use strings when we are checking/reconstructing the leading
character -- this allows for more efficient operation.

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

BUG-582: Pre-compile replacement pattern

Introduce a static Pattern, which can be used to instantiate Matchers.
Speeds up processing, which included construction of the pattern on each
call.

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

BUG-582: Sort modules on initialization

Rather than sorting them each time they are accessed, pay the price
upfront and create a sorted view of modules.

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

BUG-582: Optimize SchemaContextImpl.findModuleByNamespace()

This is by far the hottest method, taking full 10 seconds from the
startup. Optimizing it looks simple enough: just create a SetMultimap
and precompute the URI->Module mappings.

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

BUG-1070: log exceptions

When we fail to construct a schema context, we do not log any sensible
cause at all, leaving not nearly enough evidence to pin down the
problem. Step one is to start reporting thrown exceptions.

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

BUG-868: add deprecation hint

This just adds the hint for users to know where to migrate to.

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

Merge "Checkstyle logging rules"

BUG-994: introduce SchemaPath.create() and friends

This introduces a proper static constructor method, which does not
necesssarily have to allocate a new object. It also does not require an
explicit list as initializer, giving more flexibility to both users and
implementation itself.

Also introduces a couple of utility methods which make manipulating
SchemaPath easier.

This patch obsoletes the public contructor and getPath() method.

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

Bug 735 - Part 1 - Bug Fix - fix the wrong version in the ietf-restconf bundle

Change-Id: Ie70cbe61220264af7b8b805e33c4a25167f218cc
Signed-off-by: Devin Avery <devin.avery@brocade.com>

Merge "BUG-580: Improved parsing."

Merge "BUG-990: fixed deserialization of enums."

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

Merge "BUG-865: removed unused code."

Merge "Bug 571 - Throwing an exception to fail the build when the outputBaseDir is null in the pom.xml"

Merge "BUG-994: Create QNameModule for resource sharing"

Merge "Bug 735 - Part 1: Update ietf-restconf and ietf-yangtypes to newer versions"

BUG-990: fixed deserialization of enums.

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

BUG-865: removed unused code.

Removed unused imports, replaced deprecated code.

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

BUG-865: removed use of deprecated parseToClassName method.

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

Merge "BUG-509: InstanceIdentifier should be Comparable"

BUG-580: Improved parsing.

Added new YangContextParser interface.
Deprecated methods from YangModelParser.
Improved IO and exception handling.

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

Bug 571 - Throwing an exception to fail the build when the outputBaseDir is null in the pom.xml

Also modifying the pom.xmls of tests to specify the outputBaseDir

Change-Id: Ic149ee6d65c225af302eba80cb1d30469a47ade5
Signed-off-by: Mayank Agarwal <mayagarw@cisco.com>

Merge "Improved documentation of BindingReflections."

Improved documentation of BindingReflections.

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

Merge "Bug 1027: Improved instance identifier codec for augmentations"

Bug 1027: Improved instance identifier codec for augmentations

InstanceIdentifierCodecImpl
Updated instance identifier codec to also create context
for child nodes of augmentation when augmentation is
for first time encountered in new context.

Improved structure of instance identifier codec to be bit
more readable and which allows for reuse.

LazyGeneratedCodecRegistry
Updated AugmentableDispatchCodec to try serialization
of InstanceIdentifier to augmentation when new
location is discovered and report back if serialization
failed.

Added documentation to adaptForPathImpl, renamed parameters
to be more readable.

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

Bug 735 - Part 1: Update ietf-restconf and ietf-yangtypes to newer versions

Patch 2 - Implemented RevisionBuilder.java and RevisionBuilderTest.java classes.
Patch 3 - Creating new ietf-yangtypes bundle for new version of file.
Patch 4 - Renamed ietf-yangtypes bundle to full version.

Change-Id: I8824580f37869ff128fe33ea862c90261bd743a6
Signed-off-by: Devin Avery <devin.avery@brocade.com>

Checkstyle logging rules

added rules:
-avoid using printStackTrace
-avoid using System.out|err.print
-logger must be declared as "private static final"
-no more than one logger declared
-avoid using string concatenation in log message
-logger variable name must be "LOG"
-LoggerFactory#getLogger input Class type parameter is equal to wrapping class name.
-logger must be Slf4j
-log message placeholders count equals to parameters count following (excluding exception type param.)

Change-Id: I6da0c22d80de032eb047b4ca8776f2a38c4a4959
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Merge "BUG-987: improve generated type allocation"

BUG-994: Create QNameModule for resource sharing

This splits out the per-module invariants out of QName into a separate
class, QNameModule. This simplifies the QName logic quite a bit and
allows sharing of QNameModule in the future.

As a cleanup we get rid of static imports and improve exception texts.

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

BUG-994: improve object cache

This fixes the object cache to work in face of missing concrete
implementation. Also fixes up previous mistakes in cache interaction,
adding tests.

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

Move ClassLoaderUtils

This patch makes an attempt at unifying and improving the two
ClassLoaderUtils versions we have available. This is a prerequisite to
being able to being able to maintain them properly.

At this point the sanest place for them is yang-binding, as it has
guava, slf4j and is at least transitively pulled from all users.

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

BUG-1027: emit more diagnostics when augmentation is not found

This adds more stern warnings when we fail to find augmentation targets.

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

BUG-1026: do not mask unexpected exceptions

This patch restructures the code such that we do not silently drop the
exception we may encounter during module info loading -- promote it
to an IllegalArgumentException.

This has the nice feature or also not masking any RuntimeExceptions we
may encounter.

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

BUG-1026: Better message when we fail lookup

This turns a NPE into a checkArgument(), such that we emit proper
diagnostic information.

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

Fix javadoc warnings in binding-generator-impl

Adds missing description to @return, fixes parameter name.

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

Provide more information when throwin exception

This adds some context to the thrown exception such that at least the
class name is captured.

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

BUG-731: Declare which exceptions are possible from callable

This just makes it clear that it's not Exception which is being thrown.

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

Do not declare Throwable as thrown

Let's not force our callers to deal with Throwable -- Exception is bad
enough.

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

Merge "Bug 981: Make sure QName is not null in RPC case."