Code Review / mdsal.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: f29f9c5) | patch
Eliminate EntityOwnershipChange 42/109242/1
authorRobert Varga <robert.varga@pantheon.tech>
Sat, 9 Dec 2023 03:55:51 +0000 (04:55 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Sat, 9 Dec 2023 05:35:52 +0000 (06:35 +0100)
commitc91f455708a7914061c857a2399942408a4eb773
tree96a4acbed1c575a1f8422e127d92dadb196517catree | snapshot
parentf29f9c5bfe9760245f8722e4f4de3d9d60899c98commit | diff
Eliminate EntityOwnershipChange

EntityOwnershipChange is only used in ownership listeners as a single
method argument. Since it is generic on GenericEntity type, users
actually need to use @SuppressFBWarnings to safely get at the entity.

Remove the DTO and inline its contents into
GenericEntityOwnershipListener.ownershipChanged() arguments.

JIRA: MDSAL-843
Change-Id: I49f8714c030ede45b073b0d5f3db5b4c4eb5d013
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
16 files changed:
entityownership/mdsal-eos-binding-adapter/src/main/java/org/opendaylight/mdsal/eos/binding/dom/adapter/DOMEntityOwnershipListenerAdapter.java diff | blob | history
entityownership/mdsal-eos-binding-adapter/src/test/java/org/opendaylight/mdsal/eos/binding/dom/adapter/DefaultEntityOwnershipServiceTest.java diff | blob | history
entityownership/mdsal-eos-common-api/src/main/java/org/opendaylight/mdsal/eos/common/api/EntityOwnershipChange.java [deleted file] blob | history
entityownership/mdsal-eos-common-api/src/main/java/org/opendaylight/mdsal/eos/common/api/GenericEntityOwnershipListener.java diff | blob | history
entityownership/mdsal-eos-common-api/src/test/java/org/opendaylight/mdsal/eos/common/api/GenericEntityOwnershipChangeTest.java [deleted file] blob | history
entityownership/mdsal-eos-dom-api/src/test/java/org/opendaylight/mdsal/eos/dom/api/DOMEntityOwnershipChangeTest.java [deleted file] blob | history
entityownership/mdsal-eos-dom-simple/src/main/java/org/opendaylight/mdsal/eos/dom/simple/SimpleDOMEntityOwnershipService.java diff | blob | history
entityownership/mdsal-eos-dom-simple/src/test/java/org/opendaylight/mdsal/eos/dom/simple/SimpleDOMEntityOwnershipServiceTest.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/main/java/org/opendaylight/mdsal/singleton/dom/impl/AbstractClusterSingletonServiceProviderImpl.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/main/java/org/opendaylight/mdsal/singleton/dom/impl/ClusterSingletonServiceGroup.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/main/java/org/opendaylight/mdsal/singleton/dom/impl/ClusterSingletonServiceGroupImpl.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/main/java/org/opendaylight/mdsal/singleton/dom/impl/PlaceholderGroup.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/test/java/org/opendaylight/mdsal/singleton/dom/impl/AbstractDOMClusterServiceProviderTest.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/test/java/org/opendaylight/mdsal/singleton/dom/impl/ClusterSingletonServiceGroupImplTest.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/test/java/org/opendaylight/mdsal/singleton/dom/impl/DOMClusterSingletonServiceProviderAsyncImplTest.java diff | blob | history
singleton-service/mdsal-singleton-dom-impl/src/test/java/org/opendaylight/mdsal/singleton/dom/impl/DOMClusterSingletonServiceProviderImplTest.java diff | blob | history
MD-SAL: Model-Driven Service Adaptation Layer