Make LazySerializedDOMNotification a DOMEvent

DOMEvent carries the information about when the event occurred,
make sure we propagate it when available and backfill it on
submission.

It also retrofits some @NonNull annotations to make it clear we
are not propagating nulls.

JIRA: MDSAL-282
Change-Id: I2153b007af2accbad41f5aea00dbced21030a67d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize source generation

Since StringBuilder is fluent, we can optimize some of the source
generation utilities to have less bytecode. Also mark FIXMEs for
future improvement, as these would allow better devirtualization.

Furthermore we expose Class-based cast/assign operations, which
trim down code duplication a tiny bit.

Change-Id: I32d9d9321992effe0bec1423990208a92c1f780b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Define OpaqueData/OpaqueObject hashCode/equals

This adds baseline hashCode/equals specification, so that we do not
rely on identity for comparisons.

JIRA: MDSAL-439
Change-Id: I41291935a0402f6fce13b6e49531d85e2fc905d5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Tag generated QNAME field with @NonNull

Improve code generation, so that users can be sure QNAME is never
nullable, improving interactions with APIs which take a QName.

Change-Id: I0454ba448fb9edb605d04e97901381ae0f8a79d0
JIRA: MDSAL-441
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Tag InstanceIdentifier methods with @Nullable/@NonNull

Most of these methods are guaranteed to return non-null, tag them
as such, improving user experience.

Change-Id: I6f581baf2a023b90274b9d89bf6475d9780449c8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Tag NotificationPublishService.REJECTED as non-null

This can only ever be a non-null constant, tag is as such.

Change-Id: I2d011a66fa61b164dab184ab0808a77ee290c774
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump version to 4.0.1-SNAPSHOT

This starts the next release cycle.

Change-Id: Ic3462f2874e72597d6686ad25d16c607e44f7859
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Specialize LeafNodeCodecContext for TypeObjects

This adds the gue between BindingTypeObjectCodecTreeNode and
LeafNodeCodecContext, which is necessary to correctly instantiate
leaves from caches.

JIRA: MDSAL-407
Change-Id: Ia636cf513e6581ad098e0887b37119476508063a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split LeafNodeCodeContext and LeafSetNodeCodecContext

These are the two basic node contexts we are currently supporting,
split them from ValueNodeContext, properly separating deserializeObject()
implementations.

JIRA: MDSAL-436
Change-Id: Ieeb9c047f980631ab3861bb7e42ef87e52f9552e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rename LeafNodeCodecContext to ValueNodeCodecContext

'LeafNode' prefix has wrong implications, as it is reminiscent
of nodes from 'leaf' statement, when in fact it covers all nodes
considered 'leaf' values in the Binding Object tree. This covers leaf,
leaf-list, anyxml, anydata -- where leaf-list and leaf have actually
differnent handling.

This renames the class to ValueNodeCodecContext, to make a clear
distinction between the two groups.

JIRA: MDSAL-436
Change-Id: I02d60afc1650319efe6cbd90b12eeb47e5152a8f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Allow BindingNormalizedNodeCachingCodec on any BindingObject

This lowers the requirements so we can have a caching codec for
any BindingObject.

JIRA: MDSAL-407
Change-Id: I1ae3659b277df29743028761e136457c2dfd1a0f
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add BindingTypeObjectCodecTreeNode

This interface supports TypeObject codecs, exposing the appropriate
codec interface.

JIRA: MDSAL-407
Change-Id: Ib45561039b85e051086934e72daa4fa701b600aa
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split BindingNormalizedNodeCache

We will end up caching various kinds of BindingObjects, this splits
up BindingNormalizedNodeCache into an abstract base class and a
concrete implementation for DataObjects.

JIRA: MDSAL-407
Change-Id: I7ae0338fa29124cfe4a5f0234fe9dd55e8bc389e
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rework BindingCodecTreeNode

There is a strong tie-in between NodeCodecContext and
BindingCodecTreeNode, where the latter has massive oversubscription
of context.

This patch separates the semantics out, so that we end up with
cleaner interfaces and proper decomposition of concerns, so that
further binding concepts can be handled by the codec infrastructure.

This is a preparatory patch for both TypedObject caching and
Annotation/OpaqueObject translation.

JIRA: MDSAL-10
JIRA: MDSAL-285
JIRA: MDSAL-407
Change-Id: Ibc8d155b8a91b6081cd0eecd5d25fae56cb4699a
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unneeded suppressions

Now that implementedInterface() is specialized by DataObject,
we can get rid of raw types and unchecked casts, as we know
the returned Class object is a subclass of DataObject.

Change-Id: I0d148a4437462e7d05384efbb79d935b5b00e9b7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up tryToUseCacheElse()

Generated code is a bit ugly for the case of accessing cache,
with missing newline and misplaced whitespace. Clean that up.

Change-Id: I13c754c808b9da19e3f5e4cfc216073aafb462db
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Document BindingCodec/Serializer/Deserializer

These interfaces are not documented at all, owing partly to the fact
they are are implementation of mdsal-binding-dom-codec, whose design
currently requires placing codecs into the same class loader as
generated code -- hence they leak to the common binding support
package.

Change-Id: Ib576b9aeddcc644d5b1f44a2047e775cc5266245
JIRA: MDSAL-401
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix AbstractBaseType nullness handling

Use requireNonNull() instead of a custom throw, allowing us
to annotate the field as @NonNull, reducing the number of warnings.

Change-Id: I53a64abfed239c3ae76d319479916ca381595324
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update BindingMapping nullness handling

This annotates all used methods as returning @NonNull, fixing
a number of warnings in users.

Change-Id: I0d8bdf493e92ac66614755807fc34d43da38c10a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor DOMMoutPointServiceImpl UT a little bit

This refactors DOMMountPointSerivceImplTest unit test. The test now
tries to test DOMMountPointServiceImpl's DOMMountPointService API
contract instead of testing how it is implemented. This way we can
restrict the visibility of some of DOMMountPointServiceImpl members.

Change-Id: I8bd9cd7d659ffd8d32d7ce361f42c9cfec805283
Signed-off-by: Jakub Morvay <jakub.morvay@gmail.com>

Remove unneeded javadoc override

odlparent is shipping javadoc-plugin-3.1.0, we do not need to override
anymore.

Change-Id: I9cfe5cfd0786373e272773d616db78fb229dc2d2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Re-enable generation of CheckReturnValue

Instead of JSR305 we use spotbugs-annotations here.

Change-Id: I86b1c020632af2c597a48a237ff338955e2a13ea
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Reduce use of getDataChildByName()

Using findDataChildByName() is null-safe, allowing us to reduce
the number of assertions.

Change-Id: I182f52371aab5b18d94bebcd3a9aaf26b9d5fb55
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix ietf-type-util mis-bump

This version got accidentally mis-bumped during yangtools-3.0.0
-SNAPSHOT removal, fix that.

Change-Id: I9e872ced1f470a5cfdf28b7f98d54f42f92d5086
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Push out API FIXMEs

We do not have time to address all these FIXMEs, push them out
to the 5.0.0 release.

Change-Id: Ida1fb72302930a4a48eed58ff78cbf35abd9f19e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make WriteOperations.(put,merge) with boolean default methods

This inverts the API, so that implementations are required to
provide implementations of mergeParentStructure{Merge,Put} instead
of the boolean-forking put()/merge() methods, addressing associated
FIXMEs.

Change-Id: I9750982fa1ec833fda17d583e2806d4350618c1c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Hide ShardedDOMReadWriteTransactionAdapter

As per the FIXME, hide this class and make it final.

Change-Id: Ibfdbad74ab6fa16283da2ab2b109f37805e98961
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove DOMDataTreeReadWriteTransaction.close()

Inheriting from DOMDataTreeReadTransaction was a mistake, read-write
transaction should not have a close() method. Fix that by inheriting
DOMDataTreeReadOperations instead, addressing the associated FIXME.

Change-Id: Ie26e19917bd78acca056c7c79de641b231fa9ea8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make DOMDataTreeShardProducer#close() non-default

DOMDataTreeShardProducer implementations are now required to provide
a close() method, addressing the associated FIXME.

Change-Id: Ic77a34ccc0fd9af96465a807142cf0b3b5b303ab
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make MethodSignature default-related methods required

This promotes retrofitted isDefault() and setDefault() methods
to be properly required, addressing the associated FIXME.

Change-Id: Ibc958ce02c36a317f9b6f4c04632a3d6cecf0f7c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump yangtools to 3.0.0

This bumps dependencies to yangtools-3.0.0, adjusting for API changes
brought by that change:
- NormalizedNodes no longer have attributes
- DataTreeCandidateNode.getModifiedChild() returns an Optional
- ListenerRegistry is no longer an Iterable
- ClassloaderUtils is using Optional instead of nullable
- NormalizedNodeStreamWriter opens leaf/anyxml/leafsetentry nodes
- leafrefs are using PathExpression instead of RevisionAwareXPath
- path expressions are strictly validated
- QName does not allow '/'
- QueuedNotificationManager.BatchedInvoker operates on ImmutableList

Change-Id: I45b25faaea8b60de974e9ffa2b5761567115fd2a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent to 5.0.0

This bumps odlparent to 5.0.0.

Change-Id: I90398f344b45c8f55bd92831ea2221c4af34ce5e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Integrate PingPongDataBroker into DataBroker contract

The ping-pong functionality affects only transaction chains and
can be easily realized as a proxy. This patch provides
PingPongTransactionChain as an SPI utility and introduces a new
method to allocate such transaction chains.

JIRA: MDSAL-432
Change-Id: I06744a7860527d8827c280ab603f5e98077d27b6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add AbstractDOMSchemaService and FixedDOMSchemaService

AbstractDOMSchemaService allows more code reuse between implementations
by holding the common bits needed to interface with SchemaContextProviders.

Also use these to implement a FixedDOMSchemaService for use in situations
when the SchemaContext and the set of associated sources are known to
never change.

This also slightly simplifies ScanningSchemaSourceProvider.

JIRA: MDSAL-418
Change-Id: I554837f8ae93e5bbb741b4cb801c31b8ebfcaa03
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix small formatting issue

Change-Id: I48128fee6f2ff9f8e27fb5714cec1fbbd44af2e9
Signed-off-by: Jakub Morvay <jakub.morvay@gmail.com>

Do not fail if invoking of listener fails

DOMMountPointServiceImpl should not fail and propagate uncaught
exceptions from mount point listeners. When invocation of a registered
listener onMounPpointCreated or onMounPointRemoved callback fails, just
log the error and continue execution.

Change-Id: I5d182b610c0b5749cfe01efc13bc1dddfffa5283
Signed-off-by: Jakub Morvay <jakub.morvay@gmail.com>

Remove deprecated leak method

This removes already deprecated DOMMountPointServiceImpl
unregisterMountpoint leak method.

Change-Id: I6a36f0625be9657fbe5ff0337129e990a14cd6c1
Signed-off-by: Jakub Morvay <jakub.morvay@gmail.com>

Add Opaque{Data,Object} interfaces

This adds the baseline interfaces needed for generating
manifestations of anydata and anyxml node sin Binding world.

Since they are conceptually the same and cannot have the same
identifier, we can support them with a single BindingObject
sub-interface.

JIRA: MDSAL-428
Change-Id: I7d7e0ff6c7bcd7d2375d50c4e976bf68d4e47143
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add yang-binding interfaces for annotations

Binding users are interested in having a way to access RFC7952
metadata. While the actual codegen/runtime implementation requires
some work, we know what Binding Specification extension we will
need to support the generated classes.

This patch introduces Annotation and AnnotationAware, so a future
codegen version can bind to it.

JIRA: MDSAL-357
Change-Id: I2fed30ea34b21c36b626388bb37890cc73875176
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add EventInstantAware interface

This interface will be used to carry the equivalent of RFC5277
notification eventTime. It is modeled exactly the same way as
DOMEvent/DOMNotification, hence it will allow us to bridge the gap
in binding-dom-adapter, where we are losing this information.

JIRA: MDSAL-282
Change-Id: Ie93015c2609a2d1c59df6c63f76511aa68867832
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clarify DOMEvent and DOMNotification relationship

The relationship between these interfaces is realized in the
implementation, but it is not captured in the interface contract
documentation.

JIRA: MDSAL-282
Change-Id: I99d8787a16263c6628440dac8b3bd7a6bbec3f9c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Offload null value checking to CodeHelpers

Since we are using a fixed string, we can easily offload the call
to a centralized place in CodeHelpers, thus providing some minor
class size improvements.

Change-Id: I25991e7020bafc535dab6731063e39e92de03d95
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unused imports

These were introduced during MDSAL-48 implementation, remove them
again.

JIRA: MDSAL-48
Change-Id: I1fa1944bd8e1f6e0780cf2fd5d82e3b028d27a4e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Deleting a non-existent node fails

This adds a unit test for deletion of a non-existent node.

JIRA: MDSAL-108
Change-Id: I32d537c7e58a6161147e7e51b80e71cb8aaa0d79
Signed-off-by: Valentin Mayamsin <vmayamsi@cisco.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of deprecated createCursor() method

This method has been superseded by openCursor(), use that.

Change-Id: Ie64c91106c1b1cfddbd0d68456e5081f6ef72341
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BindingAdapterFactory should be Immutable

This class is immutable, which implies thread-safety, removing
the need for @ThreadSafe.

Change-Id: Ic77c7932a2dd244b0a8211bd6e8c1dc9e95bb08c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove @(Not)ThreadSafe annotation

Use documentation rather than JSR305 to document non-thread-safety.
In package-private classes these annotations often do not make
sense.

Change-Id: Iea39cf23472917e9d5765e6d2a025ff7ab7f6114
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not throw Exception

OsgiBundleScanningSchemaService.destroyInstance() does not need
to declare Exception as thrown, remove it to keep sonar happy.

Change-Id: I1bdec43b8c467f4562f33914340fa203f848c317
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Enable spotbugs in ietf-type-util

This is just a trivial flipping of the switch.

Change-Id: I916c22fee0dd9049359a044be1ee3cb6a4b673fe
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix ietf-type-util checkstyle

This fixes checkstyle violations, adding a few suppressions and
remove UnsupportedExceptionThrows and related reflection-based tests.

Change-Id: I0c1b8828a881299dde6df4fe666df000f0ec486b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix logging arguments

This is caught by upgraded spotbugs, we are missing returnType
argument.

Change-Id: I5513c89cd22e0c4fd739ada5a2d5e5b61b185a09
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Tolerate unresolvable leafrefs in groupings

In case a grouping contains a relative leafref pointing outside of
the grouping subtree we cannot safely determine the target type, which
lead to an IllegalArgumentException.

In this case we need to resolve the return type to an Object, which
loses type safety, but really is the best we can do as we just do not
know in what contexts that grouping is used.

Unfortunately this requires API changes to mdsal-binding-generator-api
as we need to communicate the fact we are resolving a type definition
in the context of a grouping to TypeProvider implementations.

JIRA: MDSAL-182
Change-Id: Iaf96d133c08eb47f4bb27b6c4f3db1463b78f05e
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Change 'type empty' mapping

This patch changes 'type empty' leaves to map to yang.common.Empty,
aligning DOM and Binding representations and removing potential
confusion around what Boolean.FALSE means in the value.

It also eliminates the need for a dedicated codec, as this type is
handled through NOOP_CODEC.

JIRA: MDSAL-48
Change-Id: If43d710d4620cffaecc8ca7d42d1eb00c58370c0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove FluentFuture from DOM RPC contracts

FluentFuture cannot be proxied in Guava 27.0, hence we need to drop
to using ListenableFuture to efficiently shortcut invocations.

Change-Id: Ib89742760e0bbfdb6c46aa4bbf99ea35f5823c8b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup RuntimeException throws

We are just wrapping exceptions in these cases, switch to wrapping
with IllegalStateException to keep sonar happy.

Change-Id: I571f444b9d97dd9d9129b6e18f7e91e72b30d842
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Speed up ClassTemplate getDefaultInstance() generation

We can skip instantiating FQCN and just compare against constants,
which results in faster dispatch.

Change-Id: I681888872db411b4f70b3b09d205540d8b607568
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Speed up Union stringValue() generation

We can compare against well-known Types, which is faster than
going through FQCN.

Change-Id: I7f6a7bd143d94be48176d9d8e1fe56bd0e6eae8e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Simplify UnionTemplate stringValue() dispatch

We have a complicated if in two cases, simplify it and take
advantage of Types.BYTE_ARRAY.

Change-Id: Ib413d5f987b0dd15cf29fbc429bde68bc35c1379
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove DataContainer.getImplementedInterface()

This method conflicts with user data and has been supplanted
by implementedInterface().

This patch removes it from codebase and instantiates a test for
the conflict.

JIRA: MDSAL-395
Change-Id: I3c2f6699a0785dbc353f3118883168b0e84407b1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Generate implementedInterface

getImplementedInterface() is trampling on getter namespace, making
it possible to clash with a user-supplied model. It also is not properly
overridden in generated code.

Fix both these issues by introducing DataContainer.implementedInterface()
and overriding it as needed wither with a narrower specification
(groupings) or a default implementation (other interfaces).

JIRA: MDSAL-396
Change-Id: I902350f0979067a9e035770e683ab18e966deb7f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove DOMMountPointServiceImpl.registerMountPoint()

This method is a leak and was deprecated in 3.0.x, remove it now in 4.0.0.

Change-Id: Ieb7c0d81872851e68dc231b5be653e782c7ed9c4
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Implement EthertypeBuilder

This implements EthertypeBuilder in an efficient way, using
pre-built known ethertypes.

It also defines {compare,equal,hash}Value, methods which perform
semantic operations, treating number-based and enumeration-based
Ethertypes equally.

Change-Id: I394f65a17fa84c06a63bc598c947d274d6dd6413
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use E$$ to capture augmentation type

'E' is a valid generated identifier, which can lead to conflicts
when generating augmentable implementations.

Change-Id: I9d69d06e18301eff788c101e389a6d0b5365ad38
JIRA: MDSAL-425
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix check path contention when create tree producers

- It's not allowed to create producers with their subtrees conflicts.
  Add a map to store producers aim to do this check more simply than
  to travel 'DOMDataTreePrefixTable'.
  It's assumed there're less producers in th map that we could accept
  to do 'for' operation.

  JIRA: MDSAL-397
Change-Id: I105291402f21b530f54873307316f0f0c5640f6b
Signed-off-by: Jie Han <han.jie@zte.com.cn>

Remove implied @ThreadSafe annotations

Immutable interface contract implies thread safety, there is no
need for these annotations.

Change-Id: I51acabd6eca13690b67a3551ae7180510a5191b0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix javadoc HTML5 compliance

This fixes up javadocs so that they are HTML5-compliant.

Change-Id: Ie62191a3235903ecba5be46a5423b878afa67553
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Fix aggregator pom name"

This reverts commit 0b318674e2b6f8bc9927960ee0577824976cacee,
as it breaks sonar reports.

Change-Id: I5d85dbb9d8ff84b200604959e060c7169c31e488
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove binding spec2

Binding spec v2 is rather incompatible and we have decided to
not make a jump to it, as it is not really feasible. Rather than
that we will backport its fixes to binding v1 in a gradual way.

Remove all binding-v2 artifacts.

Change-Id: Ic58f13372cfe6cbc6f5088e1e507acac14c11c19
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Correct docs declaration

We should not be declaring yangtools-docs, but rather mdsal-docs.

Change-Id: I1e47f3e8d6154ac29fc7d78da12e96dd07e831a9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump version for 4.0.0-SNAPSHOT

This updates snapshot artifacts for 4.0.0 release train.

Change-Id: I3900cd07963b927b34ecdd60668f47c4a3c3c85b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RFC8519 models

This adds RFC8519 ietf-access-control-list and ietf-packet-fields
models, which are replacing previous instantiation of these models.

Change-Id: I984db5e2fc13c4a4e0b12b3afe4141cf73c1d6d5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of deprecated constructors

Java 11 deprecates most of these constructurs, so let's get rid
of them.

Change-Id: I67c03370c4e8440608e0cece182d87c7ea923258
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix logging placeholders

Upgraded SpotBugs is finding these, fix them up.

Change-Id: I6209c661ccdf17d886d8c776d96fe6b6a32abc1b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix odl-mdsal-model-rfc8349 definition

This feature should be polling in rfc8344, rfc8343 and rfc6991.
Due to transitive dependencies, it is enough to pull rfc8344.

Change-Id: I1098dba897dc4be27d062925799c5be3e4432117
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RFC8530 Logical Network Elements

This adds packaging for RFC8530 model, so that it can be reused
by downstreams.

Change-Id: I863951b0caed784f5d7d6c3f5ce4d562d8755b72
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RFC8529 Network Instances

This adds packaging for RFC8529 model, so that it can be reused
by downstreams.

Change-Id: I0e6cbf9b1c45122d75b639b4be9a329cedb2a252
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RFC8528 Schema Mount model

This adds packaging for RFC8528 model, so that it can be reused
by downstreams.

JIRA: MDSAL-424
Change-Id: Ia6ec9ffd349cd6afa2629b9fa1eb9096d263aa39
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make Enumeration extend TypeObject

- By now we can use TypeObject for leaf type objects

JIRA: MDSAL-407
Change-Id: Ifc7889b154e5e8bcdcbc4885affa9d3fa88facc2
Signed-off-by: Jie Han <han.jie@zte.com.cn>

Mark typedef types as TypeObject

Make binding class of typedef implement TypeObject. In order to not
break compatibility, make sure we disregard marker interfaces when
computing serialVersionUID.

JIRA: MDSAL-406
Change-Id: I8e8bf31d3b3c43a2c843e7dd9914def12c52fee3
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup mdsal-trace-cli package

This is not part of the public contract and it encroaches on
controller's namespace. Fix that up.

Change-Id: Iae74934039ebc684ac29ef5420c4e4b2ea2d99a8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add a category for org.opendaylight.mdsal.model.ietf.util

Provide a nice description in the javadocs for the package.

Change-Id: Ic699fe849d4fdd6d7e6181a7c480d481a24cd7b1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use Collections.emptyList and Collections.emptySet

... instead of the non-generic constants.

Change-Id: I8540db4c2d52185ed0de65eb8b45b75f75cc3f91
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Use java.util.Objects.equals

... instead of com.google.common.base.Objects.equal.

Change-Id: Id4252306c443ce140e261c353dfecd601bcad84c
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Use HashMap and HashSet constructors

... instead of Maps.newHashMap and Sets.newHashSet.

Change-Id: Ie6c04a4fc546401b41943c481335e2cc320f7edd
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Switch to java.util.Base64

Change-Id: Ib944fdd09543d423478267d0ea8a386ae3efe02d
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Switch to Objects.requireNonNull

... instead of Guava’s Preconditions.checkNotNull.

Change-Id: If7979ac0071e8c1d1af2d7913c6df791942825ab
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Do not generate CheckReturnValue annotations

Use of javax.annotation is really hurting our migration to Java 11,
just do not generate it all over the place.

JIRA: MDSAL-372
Change-Id: I81aa4d8b62b12ea0ea503b5d002e744a31855aea
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add mark interface

- Add TypeObject for derived types only
- Add base BindingObject for all binding types

JIRA: MDSAL-406
Change-Id: I93ac1bea70f61a9ed05dc35951f024f08dcf6b83
Signed-off-by: Jie Han <han.jie@zte.com.cn>

Account for reported UNMODIFIED nodes

DataObjectModification has no way of expressing UNMODIFIED nodes,
but they can (and are) reported from DataTreeCandidateNode, hence
we end up reporting IllegalStateException when the user tries to
look at those nodes.

Make sure we take the node under consideration and do not report
unmodified children, hence preventing the ISE from happening.

JIRA: MDSAL-422
Change-Id: I34d36ae083c5ce3ad793eb584236f175f7a3a906
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump java-diff-utils

This upgrades java-diff-utils to version 4.0, with its new group id,
bringing better Java 9+ support and fixing a number of licensing
issues.

Change-Id: I4272f4d721dbd0af335115cc3bb2be196329ca11
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix AbstractBaseDataBrokerTest.assertCommit()

This method should reflect whatever commit() is returning, but
unfortunately it was left out during conversion.

Change-Id: I7e7389d0a7f8fbd94af2ee07ce089f72f4855a74
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Generated javadoc does not separate multi-line description

- replace any of "\n\t" with " " and then replace
  multiple " " to a single " ".

JIRA: MDSAL-416

Change-Id: I1f29f7da9f574e65a8ccf169e29883c9d1f47c26
Signed-off-by: Jie Han <han.jie@zte.com.cn>

Update junit link

javadoc.io breaks with JDK11, update the link.

JIRA: MDSAL-419
Change-Id: I9b0fdbe0d0dc888f4b5de8e25a298fb7424e5345
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

minor: fix wrong Logger in InMemoryDOMDataStoreFactory

Change-Id: I20257192bc2885a2500025bcbdf18e8fa0c8d0c6
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Update javadocs link

We have moved javadocs to a different artifact, update the link.

Change-Id: I9f81681f6b2118712b1c4cebbe0a6de5fafed194
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump to 3.0.7-SNAPSHOT

This starts the next release development.

Change-Id: I57a90dbf6d5961ea0c2d1c3bb5af5dda985132c0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix inability to promote

mdsal-docs not having the main artifact causes the repository
fail to promote. Work this around by packaging guides into a jar
rather than as a separate classifier.

Change-Id: I3c228e425da04b645f0a1a6dbd3e58cffe6e9f6f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Add BindingNormalizedNodeSerializer.coerceFromYangInstanceIdentifier"

This reverts commit 137bd301e01ecec44fa47dfbc69df664ea014c5e.

Blueprint cannot handle default methods, hence we need to revert
this, unfortunately.

Change-Id: I67917f2e5cdb70ebe45f76cae347bde209841e17
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add mdsal-docs artifact

Rather than doing javadoc:aggregate, use the docs directory to
prepare documentation, which includes sources as well as javadocs.

This cuts down the build time in half while providing a better
javadoc result due to our ability to heavily customize.

Change-Id: If39191f179265240e7a5d3700fcea32e10897220
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions to 3.0.6-SNAPSHOT

This starts the next development iteration.

Change-Id: I1012c62ad20cf9db03b8fa121db16377669e43d0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>