Code Review / openflowplugin.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 10cd656) | patch
Cleanup up registration lifecycle 05/110905/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 20 Mar 2024 13:03:55 +0000 (14:03 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 20 Mar 2024 13:17:25 +0000 (14:17 +0100)
commitac9b359c3fce7bbb99392841daf776c33a234f6d
treea8554f7bc94822578371a5cc310045ab17edbfa9tree | snapshot
parent10cd6562382140a3f7bd5ed823c8d6de5f2b6918commit | diff
Cleanup up registration lifecycle

We have a StackOverflowError during shutdown caused by recursive
invocation. This stems from a lifecycle misunderstanding.

Remove close() methods and use plain yangtools.concepts.Registration
to have clear semantics.

Change-Id: I356f9f5e7bd179799b7ca5e081714d02d4ee0903
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
12 files changed:
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/DeviceMastershipManager.java diff | blob | history
applications/reconciliation-framework/src/main/java/org/opendaylight/openflowplugin/applications/reconciliation/impl/ReconciliationManagerImpl.java diff | blob | history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/mastership/MastershipChangeRegistration.java diff | blob | history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/mastership/MastershipChangeService.java diff | blob | history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/mastership/MastershipChangeServiceManager.java diff | blob | history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/mastership/ReconciliationFrameworkEvent.java diff | blob | history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/mastership/ReconciliationFrameworkRegistration.java diff | blob | history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/mastership/MastershipChangeServiceManagerImpl.java diff | blob | history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/mastership/MastershipServiceDelegate.java diff | blob | history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/mastership/ReconciliationFrameworkServiceDelegate.java [deleted file] blob | history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/lifecycle/ContextChainHolderImplTest.java diff | blob | history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/mastership/MastershipChangeServiceManagerImplTest.java diff | blob | history
OpenFlow Plugin