minor: add TODO re AutoCloseable in MdsalModule
authorMichael Vorburger <mike@vorburger.ch>
Sat, 22 Dec 2018 01:17:16 +0000 (02:17 +0100)
committerMichael Vorburger <mike@vorburger.ch>
Wed, 1 Jul 2020 23:38:11 +0000 (01:38 +0200)
Signed-off-by: Michael Vorburger <mike@vorburger.ch>
src/main/java/org/opendaylight/mdsal/simple/MdsalModule.java

index 00a1b00a3e05c2296d2f5c21bf4e1f90a9b55c16..5456a19416235c0ca5d82e8cf820db63fb45fd66 100644 (file)
@@ -31,6 +31,9 @@ import org.opendaylight.mdsal.singleton.dom.impl.DOMClusterSingletonServiceProvi
 @SuppressWarnings("deprecation") // sure, but that's the point of this class...
 public class MdsalModule extends AbstractModule {
 
+    // TODO [LOW] like InMemoryControllerModule extends AbstractCloseableModule and close up..
+    // or rather, better, instead just annotate respective classes with @PreDestroy!
+
     @Override
     protected void configure() {
         // TODO This is WRONG; later need to use the DistributedEntityOwnershipService instead here!