DataBrokerTestModule getBindingToNormalizedNodeCodec() & Co.

need these exposed for a new standalone component style test  I'm on

JIRA: MDSAL-556
Change-Id: I5efd3a6e507d46d035067a561aaca7c2f80f51e6
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 8163: getDataTreeChangeListenerExecutor() & DataBrokerTestModule

Adjust AbstractDataBrokerTestCustomizer with a
getDataTreeChangeListenerExecutor() instead of a
setDataTreeChangeListenerExecutor(), just for more consistency with the
existing getCommitCoordinatorExecutor() method.  Also less confusing (to
me) than seeing the private Executor set by default which may get
changed by the setter later.

Adjust ConcurrentDataBrokerTestCustomizer with the
useMTDataTreeChangeListenerExecutor as a constructor argument, instead
of an useMTDataTreeChangeListenerExecutor() method, just for consistency
for how you already have it in AbstractConcurrentDataBrokerTest.

Extend ConstantSchemaAbstractDataBrokerTest and DataBrokerTestModule to
allow passing through this new opt-in tweak flag, so that tests in
downstream projects such as genius and netvirt can staring exploring
enabling this.

JIRA: MDSAL-556
Change-Id: I4ad85ac48163d2f4bac865f46a3b047d5b7d333a
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

AbstractConcurrentDataBrokerTest @deprecate-s the AbstractDataBrokerTest

This is take #2 on a similar earlier attempt which made the same change
directly in AbstractDataBrokerTest, which broke some downstrean tests,
which were BADLY WRITTEN (!) because the assumed single threaded direct
execution (which DataBroker does not guarantee).

https://bugs.opendaylight.org/show_bug.cgi?id=7538 has full background

JIRA: MDSAL-556
Change-Id: I05ac3525bdcf1ab9c99dfa15b98e090848d0fddc
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

AbstractDataBrokerTestTest which actually does something

Incl. verification that each @Test cleans up after itself.  We had a
suspicion that this didn't work; it turns out that it actually does and
we had another problem somewhere else.  Contributing this test still
seems useful for future non-regression.

JIRA: MDSAL-556
Change-Id: I295ebde45a0c030ab74852788b45203cdf6639e0
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

DataBrokerTestModule: use AbstractDataBrokerTest without inheritance

While starting to write JUnit tests (for functional components) with the
AbstractDataBrokerTest, I found myself not liking its design which
forces one to use a class SomeTest extends AbstractDataBrokerTest...

Modern JUnit tests should be written by composition, not enforced
inheritance.  Thus this new DataBrokerTestModule allows one to obtain a
DataBroker suitable for Tests from anywhere.

The *(Test)Module naming convention is obviously inspired by a DI point
of view (think Spring Framework's @Configuration classes, or Google
Guice or Dagger (yay!) *Module classes - which is exactly what this
really is - a particular way to obtain an instance of an
implementationof some service (here a DataBroker) in a certain
environment (here for tests).

The internal implementation of DataBrokerTestModule could be changed
later; I guess ideally what's in AbstractDataBrokerTest could go into
DataBrokerTestModule and AbstractDataBrokerTest could use it, but for
now I'm too lazy to change that, as this does the trick nicely.  Later
implementation changes would be transparent to users of
DataBrokerTestModule.

JIRA: MDSAL-556
Change-Id: I9641f527bbc0cb92732f2e513cdd64cc6a837200
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix SnapshotBackedWriteTransaction error message

The message is misleading, as we are not performing a write, but
a merge operation.

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

Do not use raw superclass for CodecDataObject instances

We can improve generated class accuracy by specifying generic type
argument of CodecDataObject, thus making generated classes look
not like raw subclasses.

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

Add test-sources for mdsal-binding-dom-adapter

We are (unwisely) constructing a test-jar here, hence we should
also be publishing the corresponding sources.

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

Add lazily-instantiated maps

Instantiating an ImmutableMap on each access is quite wasteful,
as there are a number of cases where we do not need the entire
mapping to be instantiated.

Typical uses touch the Map only once -- either to lookup a value
based on the key, or to iterate through the values. Others are
possible, but those two cover most of the patterns.

We add the prerequisite indirection and two implementations to
support each of the cases. The iterator part is optimized to
perform lazy object instantiation -- similar to LazyBindingList.

The lookup part works in a rather similar way, except it keeps
a lookup ConcurrentHashMap as the primary storage and values
are seen as expendable secondary storage. This makes iteration
order unstable, which might be seen as a violation of immutable
contract -- but our contract is a Map and mappings remain stable.

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

Add lazily-instantiated lists

Lists are usually searched linearly. For large lists we can end
up instantiating a large number of items upfront, some of which is
wasteful.

The cutoff point can be configured via a JVM property, which is
interpreted as a dynamic constant.

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

Guarantee non-null in CodecDataObject.codecKey()

The key loaded is guaranteed to be non-null, propagate that
annotation outwards.

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

Eliminate KeyedListNodeCodecContext.Unordered.getKey()

This method and the methodhandle invoker are completely superfluous,
as we can directly access key() from the created object. While capturing
a MethodHandle lowers what method can be invoked, it does not really
buy us much, as the observed class will we always the same.

That property is visible enough (wired through an invokeExact() on the
constructor), hence JIT should be able to optimize without this help.

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

Take advantage of MapNode.size()

Checking the size allows us to skip instantiating values, plus we
can pass it down to map construction, where a correct strategy
can be taken based on the size.

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

Remove semicolons in comments

sonarcloud insists this is commented-out code, which it is for
documentation purposes. Eliminating the trailing semicolon fools
it just enough to not notice.

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

Add InstanceIdentifier.verifyTarget()

There are a number of places where users are casting InstanceIdentifiers,
just to restore type-safety after a type-unsafe operation, such as
going through serialization. Add InstanceIdentifier.verifyTarget(),
which restores type safety.

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

Add odl-mdsal-trace artifact declaration

Since controller was encroaching on this space and its version
did cover our functionality this was never packaged. Now that
controller does not package it, we need to do that ourselves.

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

Add datastore benchmark

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

Improve notification delivery defensiveness

If a listener throws an exception, make sure to log it and proceed
as if nothing happened.

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

Fix unsubscribe checks in DOMNotificationRouterEvent

The checks here are based on old assumption that the listener
would be set to null, which does not hold true for quite some
time. Update the check to close a simple race.

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

Bump versions to 6.0.1-SNAPSHOT

This starts the next development iteration.

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

Allow DOMMountPoint to not have a model context

We need to allow mount points without a context, as that is what
NETCONF requires for now.

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

Revert "Remove deprecated uint migration elements"

This reverts commit 1382c1d88b753e152ff26eae1091cd42e5aba76b,
reinstating generation of compatibility elements.

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

Bump to yangtools-5.0.1

We need further changes to tie together netconf and us.

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

Revert "Update RPC invocation to take ContainerNode"

This reverts commit f77a6182a4af3150fac68911cc98d0259342136d, as
we cannot tolerate ContainerNode in arguments either, due to how
netconf is implemented.

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

Revert "Turn DOMRpcResult's result back to NormalizedNode"

This reverts commit db7aba121dd8f6b4ee793325a31d91e28e0185d3, as
we are backing things out.

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

Turn DOMRpcResult's result back to NormalizedNode

Eventhough we really want to have ContainerNode here, NETCONF
is utterly misusing normalization to tunnel AnyXmlNode here.

Revert the change donewwwe have done, so that it continues to
work.

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

Catch nulls in RpcServiceAdapter

Binding/DOM invocation should not tolerate null input, make sure
we stop such attempts early.

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

Cleanup BindingRuntimeHelpers

We need to differentiate between testing (static context) and dynamic
contexts. This makes it possible to use these helpers in OSGi if we
inject the right set of services.

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

Generate legacy List adaptation

In order to lower the pain of migration, generate legacy setFoo()
methods with List argument. These internally perform translation
to an ImmutableMap with unique index.

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

Further deprecate addAugmentation()

We want to remove this method in next major release. Make sure
its deprecated for removal.

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

Remove AbstractBindingRuntimeContext.toString()

BindingRuntimeTypes can have a huge string representation, which
hurts when combined with a subclass of it being present in OSGi SR
and bundle-diag printing out services.

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

Update RPC invocation to take ContainerNode

Our acceptance of NormalizedNode here is misleading, we should be
taking a ContainerNode, just as actions do.

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

OSGiDOMSchemaService should provide YANG sources

This was caught in downstream integration: the refactor here
forgo to add routing towards YANG source.

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

Update BindingRuntime{Context,Generator,Types}

These interfaces should be operating on EffectiveModelContext to
provide smooth transition to downstreams. Fix that, along with
implementation and tests.

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

Adjust DOMMountPoint to be EffectiveModelContextProvider

SchemaContext does not cut it here. We really should not have this
at all and should rely on DOMSchemaService being present, but that
refactor has to wait for another breaking release.

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

Reduce use of BaseTemplate.getterMethodName()

We are using BuilderGeneratedProperty in Builder(Impl)Template,
and this class has access to the getter method. We therefore do
not need to construct the getter name, but rather can access it
directly.

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

Deprecate misnamed FooBuilder.addAugmentation()

Add explicit documentation to generated methods and deprecate
the two-argument addAugmetation() variant. Steer users towards
the one-argument variant or removeAugmentation().

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

Clean up GeneratedProperty use

Previous patch made a bit of a mess in GeneratedProperty. Clean this
up by creating a dedicated BuilderGeneratedProperty to hold the
properties created in java-api-generator.

Also add a few dedicated tests to ensure generated builders what
they are supposed to do.

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

Expand ValueMechanics with NONNULL

Since we have introduced a backchannel to add semantics, this can
be (mis)used to communicate nullness guarantees. Add another
ValueMechanics which can be recognized as needed.

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

Squash empty lists/maps

Dealing with lists requires us to interfere with what the user
is giving us in builders, effectively creating new mechanics.

This patch adds the model metadata to carry these around, exposing
just how badly broken some binding.model.api construcs are misused.
We do not fix the misuse, but make it slightly worse, but also mark
the mess for future cleanup.

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

Centralize non-key property copies

AbstractBuilderTemplate can discern between a copied key property
and other properties. Make sure we split the second part into a
separte method, allowing it to be customized.

The immediate benefit here is that we do not end up copying properties
if there are no keys.

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

Fixup DataObjectCodecContext.createBindingProxy() nullness

This method cannot return a null, nor can anything which ultimately
uses it as its sole return.

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

Squash empty MapNode/UnkeyedListNode objects to null

When we encounter an empty keyed/unkeyed list, we need to treat
it as it were not present. This adjusts the appropriate codecs
to ensure that is the case.

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

Simplify builder's addAugmentation() method

All augmentation already extend DataObject, which really is to say
that Augmentation should be doing the same. This has the benefit
of giving us access to DataContainer.implementedInterface(), which
we can use to add a simplified addAugmentation() method.

JIRA: MDSAL-183
Change-Id: I053a2d23e6bf95e07fa58e4078a37661e6e726a3
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove explicit UOE throws

Private constructors do not really have to be that defensive.
Remove exception throws and related tests.

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

Add CodeHelper.emptyToNull()

We will need to be squashing empty collections to nulls, add
utility methods to do that.

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

Remove CodeHelpers.nonNullValue()

This method is used only with augmentations. Remove it and update
callers to use Objects.requireNonNull() instead.

This removes a source of IAEs in favor of more consistent NPE
as well as trims down a few bytes from builder classes.

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

Add CodeHelper documentation

We are missing documentation for generic method types, add a few
words.

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

Improve often-used class imports

importedName(Class) is slightly slower than importName(JavaTypeName),
hence improve performance by creating a few more often-used constants.

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

Reorder encapsulation checks

Reordering checks provides a small speed up due to fewer checks
being made on average, as we take care of most popular types first.

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

Speed up BaseYangTypesProvider's restricted types

We are currently bouncing through explicit classes and a switch
expression which pretty-much duplicates BaseYangTypes. Clean this
up by exposing an alternative method to create restricted types,
so that we can first lookup mapping and then create a restricted
type.

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

Improve BaseYangTypes.javaTypeForYangType()

This method should not tolerate nulls and it can be implemented
in terms of a simple switch() statement instead of a lookup map.

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

Clean up test naming confusion

BaseTypeProvider is really a BaseTypesTest, and that test is really
BaseYangTypesProviderTest. Clean them up, relocating them as to their
home packages.

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

Speed up BaseYangTypesProvider

Most of the time we do not need to go through the switch expression
and restricted types. Short-circuit lookup when we know we do not
have restrictions.

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

Correct ClassTemplate's check for types

ClassTemplate's checks run against Java Types, which is fine, except
that we should be referencing them through BaseYangTypes, so that we
have the incidental alignment well covered.

Furthermore the checks rely on FQCN, which is potentially slow to
compute, while the new checks rely on plain type equality -- hence
potentially speeding up things and lowering the number of branches.

We also drop a FIXMEs to clarify situation around InstanceIdentifier.

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

Correct UnionTemplate's check for types

UnionTemplate's checks run against Java Types, which is fine, except
that we should be referencing them through BaseYangTypes, so that we
have the incidental alignment well covered.

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

Move BaseYangtypes to binding.model.util

These constants sit between Types and BindingTypes, hence they should
not be in a completely different package.

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

Split out BaseYangTypesProvider

BaseYangTypes is related to binding.model.api, while the type provider
is related to binding.generator.api. Split out
BaseYangTypes.BASE_YANG_TYPES_PROVIDER as a BaseYangTypesProvider.INSTANCE,
separating these two concerns.

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

Cleanup java-api-generator Constants

We have a number of useless constants here, remove them along with
the not-so-useful test.

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

Enforce non-null input on RPCs

RPC invocations should never have a null input, rather an empty
container.

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

Tag TypeObjects which wrap a value with ScalarTypeObject

Capturing this type of objects has a nice side-effect of being
able to specialize code generation. It is furthermore useful
for providing a common definition of getValue() method.

This is an unfortunate side-effect of changing the API
mapping for boolean typedefs: the accessor changes from isValue()
to getValue().

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

Remove executable bits

Plain files should not be executable, correct that mistake.

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

Bump yangtools to 5.0.0

Adopt released yangtools-5.0.0, ending snapshot integration.

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

Improve grouping sort allocation

Pre-allocating the result list saves us the common case when
the input is actually empty.

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

Improve grouping error reporting

Reporting FQCN here is useful, as it is giving us crucial information
as to what class is being generated -- including the namespace.

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

Remove odl-mdsal-model-draft-ietf-netmod-acl-model-07

These models come from a draft which has been published as RFC8519.
As we are shipping the standardized model, remove the old draft
reducing associated baggage.

The only downstream user is netvirt's aclservice, which will have
these models integrated.

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

Remove odl-mdsal-model-draft-ietf-netmod-iana-afn-safi-00

This is a long-expired draft which is used only by lispflowmapping.
Remove it from MD-SAL as part of migration to its sole user.

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

Remove ietf-lisp-address-types

This model is ancient and is used only in one of our downstreams,
lispflowmapping. Remove it along with its packaging feature.
This patch has a mirror in lispflowmapping, which reintroduces
the two files removed in that project.

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

BUG-3128: make context reference mandatory

It does not make quite sense to have an optional context
reference: the RPC is either routed (action, with a reference)
or not. If the leaf is not mandatory, there is is a mismatch
between providers and consumers with regard to what is it
that invoking an action as an RPC really means.

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

Switch DOMNotificationRouter to OSGi DS

DOMNotificationRouter is easily converted to declarative services,
this performs the migration.

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

Deprecated getSessionContext() for removal

We really want to be removing this method, mark it for removal.

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

Add Binding/DOM Query language adapter

Provide implementation of binding language translation on top
of its DOM counterpart.

On the DOM layer the expression can be transmitted, hence it gives
the possibility to move the execution to storage backend, thus
reducing app/backend data interchange volume.

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

Split DOMRpcRouter from blueprint

Add OSGi DS wiring instead of blueprint, so that we can ditch
some references to ODL extensions. This also neccessitates adding
a few missing forwarding implementations.

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

Fixup odl-mdsal-trace's name

Fix naming here to be consistent with other features by adding
"MD-SAL" infix.

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

Refactor binding-dom-adapter

BindingNormalizedNodeCodecRegistry is a serializer which allows
dynamic updates to its backing codec. Rather than referencing this
implementation, reference either BindingNormalizedNodeSerializer
(interface) or BindingCodecContext, which is the simple implentation.

This makes it obvious we are not using BindingNormalizedNodeCodecRegistry
in the context of mdsal-binding-dom-codec at all, making it a good
candidate to move to mdsal-binding-dom-adapter. Except we do not want
to just move it, but refactor so that it is properly controlled.

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

Add mdsal-binding-dom-codec-osgi dependency

docs should depend on this artifact to have proper build order
and complete javadocs.

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

Hide BindingMountPointAdapter

This class is an internal implementation detail, hide it. Also
use Codec directly rather than taking a registry reference.

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

Fix a BindingToNormalizedNodeCodec warning

We are not using proper generic arguments, which the interface
specification requires. Fix that.

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

Remove BindingToNormalizedNodeCodec.getCodecFactory()

This is a duplicate of getCodecRegistry(), remove it.

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

Improve verification message

Since we have the class identity available, let's use it to diagnoze
failures to supply a proper resource name.

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

Remove BindingToNormalizedNodeCodec.instanceIdentifierToNode()

This method has a single in-package user, move it to centralize
logic and reduce codec clutter.

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

Remove BindingToNormalizedNodeCodec.toNormalized(Entry)

This method is a simple delegator to a interface-specified method
with a single caller -- refactor the caller and remove it.

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

Remove BindingToNormalizedNodeCodec.toNormalized(InstanceIdentifier)

This method is a duplicate of
BindingNormalizedNodeSerializer.toYangInstanceIdentifier(), remove it
and adjust callers to reduce confusion.

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

Fix mdsal-binding-dom-codec exports

We only need the .impl package exported, as we do not want to
embed our -api artifact, as that just wrecks source code lookups.

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

Remove BindingToNormalizedNodeCodec.toBinding()

This method is not used anywhere, remove it to keep our codebase
free of distractions.

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

Remove AbstractForwardedDataBroker

This class hosts a chunk of functionality which is not used anymore,
dating back to when we had DataChangeListeners. Remove it.

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

Remove BindingToNormalizedNodeCodec.getNotificationClasses()

This method is not used anywhere, remove it to reduce the amount
of untested legacy code.

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

Add more error context to ResourceYangModuleInfo

In case we end up failing to load a resource, log the subclass
and potentially the ClassLoader where the problem occured.

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

Cache YangModuleInfo type name

Analysis of a heap dump shows we are creating a number of instances
pointing to the same type for the purposes of having the QNAME constant.

As we have the context in which we are creating YangModuleInfo,
instantiate a primitive cache to de-duplicate these definitions.

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

Unable to resolve grouping test

JIRA MDSAL-531
Change-Id: Icdba0c0babf93ca14f7d81b42cc432eb4455d6e0
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Make DOMSchemaService operate of EffectiveModelContext

We are stepping towards working on Declared/EffectiveStatements
instead of SchemaNodes, this exposes EffectiveModelContext from
the DOMSchemaService.

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

Bump odlparent to 7.0.1

This picks up latest updates.

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

Remove WriteOperations.put()/merge() with ensure parents

These methods have been deprecated in favor of their explicit
counterparts, mergeParentStructurePut() and mergeParentStructureMerge().

While all users have not been migrated, force that migration to
happen now.

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

Fix logic ensuring parent presence

When we are asked to create intermediate nodes for a path, whose
parent is the conceptual root (i.e. its parent is empty), we should
not attempt to merge it, as it just does not make sense.

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

Remove superfluous @NonNull annotations

These annotations are inherited from interface contract, there is
no point in specifying them again.

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

Migrate test to use mergeParentStructure{Merge,Put}

We have deprecated these methods, make sure we migrate the test
so that it does not add false warnings.

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

Realign junit links

Align JUnit links with odlparent-7, so that we generate nice
javadocs.

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

Bump byte-buddy to 1.10.9

https://github.com/raphw/byte-buddy/releases/tag/byte-buddy-1.10.8
https://github.com/raphw/byte-buddy/releases/tag/byte-buddy-1.10.9

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

Fixup netty's parent reference

We want to pull in 7.0.0, not 6.0.3.

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

Add a simple Binding query language

Offloading some amount of data matching towards the storage engine
can give use two-fold improvement:

1) skip instantiation of Binding DTOs, as the query can be realized
   in terms of DOM matches
2) potentially skip transfer of large amount of data

For Binding layer we want to have something type-safe, where we are
in control of the flow the language.

This provides a first cut at the structure, which may end up being
further evolved.

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