X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsamples%2Fclustering-test-app%2Fprovider%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fclustering%2Fit%2Fprovider%2FMdsalLowLevelTestProvider.java;h=10dafba452be749dbafd447f93c70b65ccd2f1e3;hb=HEAD;hp=27c698da174b43b81a5903a8ef34727aef60a1f8;hpb=e84f63ee098fff5b02cbce1281ca0d1208f966fa;p=controller.git diff --git a/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/MdsalLowLevelTestProvider.java b/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/MdsalLowLevelTestProvider.java index 27c698da17..10dafba452 100644 --- a/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/MdsalLowLevelTestProvider.java +++ b/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/MdsalLowLevelTestProvider.java @@ -20,7 +20,9 @@ import java.util.Optional; import java.util.concurrent.ExecutionException; import java.util.concurrent.TimeUnit; import java.util.concurrent.TimeoutException; -import org.opendaylight.controller.cluster.ActorSystemProvider; +import javax.annotation.PreDestroy; +import javax.inject.Inject; +import javax.inject.Singleton; import org.opendaylight.controller.cluster.datastore.DistributedDataStoreInterface; import org.opendaylight.controller.cluster.datastore.utils.ActorUtils; import org.opendaylight.controller.cluster.raft.client.messages.Shutdown; @@ -38,92 +40,119 @@ import org.opendaylight.mdsal.binding.api.RpcProviderService; import org.opendaylight.mdsal.binding.dom.codec.api.BindingNormalizedNodeSerializer; import org.opendaylight.mdsal.common.api.LogicalDatastoreType; import org.opendaylight.mdsal.dom.api.DOMDataBroker; -import org.opendaylight.mdsal.dom.api.DOMDataTreeChangeListener; -import org.opendaylight.mdsal.dom.api.DOMDataTreeChangeService; +import org.opendaylight.mdsal.dom.api.DOMDataBroker.DataTreeChangeExtension; import org.opendaylight.mdsal.dom.api.DOMDataTreeIdentifier; import org.opendaylight.mdsal.dom.api.DOMDataTreeReadTransaction; -import org.opendaylight.mdsal.dom.api.DOMRpcImplementationRegistration; import org.opendaylight.mdsal.dom.api.DOMRpcProviderService; import org.opendaylight.mdsal.dom.api.DOMSchemaService; -import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider; -import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceRegistration; +import org.opendaylight.mdsal.singleton.api.ClusterSingletonServiceProvider; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.AddShardReplica; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.AddShardReplicaInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.AddShardReplicaOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.CheckPublishNotifications; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.CheckPublishNotificationsInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.CheckPublishNotificationsOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.CheckPublishNotificationsOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.IsClientAborted; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.IsClientAbortedInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.IsClientAbortedOutput; -import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.OdlMdsalLowlevelControlService; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterBoundConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterBoundConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterBoundConstantOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterBoundConstantOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterConstantOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterConstantOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterDefaultConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterDefaultConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterDefaultConstantOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterFlappingSingleton; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterFlappingSingletonInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterFlappingSingletonOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterFlappingSingletonOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterSingletonConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterSingletonConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterSingletonConstantOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RegisterSingletonConstantOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RemoveShardReplica; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RemoveShardReplicaInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.RemoveShardReplicaOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.ShutdownShardReplica; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.ShutdownShardReplicaInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.ShutdownShardReplicaOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.ShutdownShardReplicaOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.StartPublishNotifications; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.StartPublishNotificationsInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.StartPublishNotificationsOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.StartPublishNotificationsOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDdtl; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDdtlInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDdtlOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDtcl; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDtclInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDtclOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeDtclOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeYnl; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeYnlInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeYnlOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.SubscribeYnlOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterBoundConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterBoundConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterBoundConstantOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterBoundConstantOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterConstantOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterConstantOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterDefaultConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterDefaultConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterDefaultConstantOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterFlappingSingleton; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterFlappingSingletonInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterFlappingSingletonOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterFlappingSingletonOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterSingletonConstant; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterSingletonConstantInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterSingletonConstantOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnregisterSingletonConstantOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDdtl; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDdtlInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDdtlOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDtcl; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDtclInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDtclOutput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeDtclOutputBuilder; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeYnl; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeYnlInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.UnsubscribeYnlOutput; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.WriteTransactions; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.WriteTransactionsInput; import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.control.rev170215.WriteTransactionsOutput; -import org.opendaylight.yangtools.concepts.ListenerRegistration; +import org.opendaylight.yang.gen.v1.tag.opendaylight.org._2017.controller.yang.lowlevel.target.rev170215.IdSequence; +import org.opendaylight.yangtools.concepts.AbstractObjectRegistration; import org.opendaylight.yangtools.concepts.ObjectRegistration; +import org.opendaylight.yangtools.concepts.Registration; import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; import org.opendaylight.yangtools.yang.common.ErrorTag; import org.opendaylight.yangtools.yang.common.ErrorType; import org.opendaylight.yangtools.yang.common.RpcResult; import org.opendaylight.yangtools.yang.common.RpcResultBuilder; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; +import org.osgi.service.component.annotations.Activate; +import org.osgi.service.component.annotations.Component; +import org.osgi.service.component.annotations.Deactivate; +import org.osgi.service.component.annotations.Reference; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import scala.concurrent.duration.FiniteDuration; -public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlService { +@Singleton +@Component(service = {}) +public final class MdsalLowLevelTestProvider { private static final Logger LOG = LoggerFactory.getLogger(MdsalLowLevelTestProvider.class); - private final ObjectRegistration registration; + private final Registration registration; private final DistributedDataStoreInterface configDataStore; private final BindingNormalizedNodeSerializer bindingNormalizedNodeSerializer; private final DOMDataBroker domDataBroker; @@ -131,34 +160,30 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS private final NotificationService notificationService; private final ClusterSingletonServiceProvider singletonService; private final DOMRpcProviderService domRpcService; - private final DOMDataTreeChangeService domDataTreeChangeService; + private final DataTreeChangeExtension dataTreeChangeExtension; - private final Map, DOMRpcImplementationRegistration> - routedRegistrations = new HashMap<>(); - - private final Map> ynlRegistrations = new HashMap<>(); + private final Map, Registration> routedRegistrations = new HashMap<>(); + private final Map> ynlRegistrations = new HashMap<>(); + private final Map publishNotificationsTasks = new HashMap<>(); - private DOMRpcImplementationRegistration globalGetConstantRegistration = null; - private ClusterSingletonServiceRegistration getSingletonConstantRegistration; + private Registration globalGetConstantRegistration = null; + private Registration getSingletonConstantRegistration; private FlappingSingletonService flappingSingletonService; - private ListenerRegistration dtclReg; + private Registration dtclReg; private IdIntsListener idIntsListener; - private final Map publishNotificationsTasks = new HashMap<>(); + @Inject + @Activate public MdsalLowLevelTestProvider( - // FIXME: do not depend on this service - final RpcProviderService rpcRegistry, - final DOMRpcProviderService domRpcService, - final ClusterSingletonServiceProvider singletonService, - // FIXME: do not depend on this service - final DOMSchemaService schemaService, - final BindingNormalizedNodeSerializer bindingNormalizedNodeSerializer, - final NotificationPublishService notificationPublishService, - final NotificationService notificationService, - final DOMDataBroker domDataBroker, - final DistributedDataStoreInterface configDataStore, - // FIXME: do not depend on this service - final ActorSystemProvider actorSystemProvider) { + @Reference final RpcProviderService rpcRegistry, + @Reference final DOMRpcProviderService domRpcService, + @Reference final ClusterSingletonServiceProvider singletonService, + @Reference final DOMSchemaService schemaService, + @Reference final BindingNormalizedNodeSerializer bindingNormalizedNodeSerializer, + @Reference final NotificationPublishService notificationPublishService, + @Reference final NotificationService notificationService, + @Reference final DOMDataBroker domDataBroker, + @Reference final DistributedDataStoreInterface configDataStore) { this.domRpcService = domRpcService; this.singletonService = singletonService; this.bindingNormalizedNodeSerializer = bindingNormalizedNodeSerializer; @@ -167,14 +192,42 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS this.domDataBroker = domDataBroker; this.configDataStore = configDataStore; - domDataTreeChangeService = domDataBroker.getExtensions().getInstance(DOMDataTreeChangeService.class); + dataTreeChangeExtension = domDataBroker.extension(DataTreeChangeExtension.class); + + registration = rpcRegistry.registerRpcImplementations( + (UnregisterSingletonConstant) this::unregisterSingletonConstant, + (StartPublishNotifications) this::startPublishNotifications, + (SubscribeDdtl) this::subscribeDdtl, + (WriteTransactions) this::writeTransactions, + (IsClientAborted) this::isClientAborted, + (RemoveShardReplica) this::removeShardReplica, + (SubscribeYnl) this::subscribeYnl, + (UnregisterBoundConstant) this::unregisterBoundConstant, + (RegisterSingletonConstant) this::registerSingletonConstant, + (RegisterDefaultConstant) this::registerDefaultConstant, + (UnregisterConstant) this::unregisterConstant, + (UnregisterFlappingSingleton) this::unregisterFlappingSingleton, + (AddShardReplica) this::addShardReplica, + (RegisterBoundConstant) this::registerBoundConstant, + (RegisterFlappingSingleton) this::registerFlappingSingleton, + (UnsubscribeDdtl) this::unsubscribeDdtl, + (UnsubscribeYnl) this::unsubscribeYnl, + (CheckPublishNotifications) this::checkPublishNotifications, + (ShutdownShardReplica) this::shutdownShardReplica, + (RegisterConstant) this::registerConstant, + (UnregisterDefaultConstant) this::unregisterDefaultConstant, + (SubscribeDtcl) this::subscribeDtcl, + (UnsubscribeDtcl) this::unsubscribeDtcl); + } - registration = rpcRegistry.registerRpcImplementation(OdlMdsalLowlevelControlService.class, this); + @PreDestroy + @Deactivate + public void close() { + registration.close(); } - @Override @SuppressWarnings("checkstyle:IllegalCatch") - public ListenableFuture> unregisterSingletonConstant( + private ListenableFuture> unregisterSingletonConstant( final UnregisterSingletonConstantInput input) { LOG.info("In unregisterSingletonConstant"); @@ -197,8 +250,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS } } - @Override - public ListenableFuture> startPublishNotifications( + private ListenableFuture> startPublishNotifications( final StartPublishNotificationsInput input) { LOG.info("In startPublishNotifications - input: {}", input); @@ -212,8 +264,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(new StartPublishNotificationsOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> subscribeDtcl(final SubscribeDtclInput input) { + private ListenableFuture> subscribeDtcl(final SubscribeDtclInput input) { LOG.info("In subscribeDtcl - input: {}", input); if (dtclReg != null) { @@ -224,31 +275,27 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS idIntsListener = new IdIntsListener(); - dtclReg = domDataTreeChangeService.registerDataTreeChangeListener( - new DOMDataTreeIdentifier(LogicalDatastoreType.CONFIGURATION, WriteTransactionsHandler.ID_INT_YID), + dtclReg = dataTreeChangeExtension.registerTreeChangeListener( + DOMDataTreeIdentifier.of(LogicalDatastoreType.CONFIGURATION, WriteTransactionsHandler.ID_INT_YID), idIntsListener); return RpcResultBuilder.success(new SubscribeDtclOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> writeTransactions(final WriteTransactionsInput input) { + private ListenableFuture> writeTransactions(final WriteTransactionsInput input) { return WriteTransactionsHandler.start(domDataBroker, input); } - @Override - public ListenableFuture> isClientAborted(final IsClientAbortedInput input) { + private ListenableFuture> isClientAborted(final IsClientAbortedInput input) { return null; } - @Override - public ListenableFuture> removeShardReplica( + private ListenableFuture> removeShardReplica( final RemoveShardReplicaInput input) { return null; } - @Override - public ListenableFuture> subscribeYnl(final SubscribeYnlInput input) { + private ListenableFuture> subscribeYnl(final SubscribeYnlInput input) { LOG.info("In subscribeYnl - input: {}", input); if (ynlRegistrations.containsKey(input.getId())) { @@ -258,21 +305,25 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS .buildFuture(); } - ynlRegistrations.put(input.getId(), - notificationService.registerNotificationListener(new YnlListener(input.getId()))); + final var id = input.getId(); + final var listener = new YnlListener(id); + final var reg = notificationService.registerListener(IdSequence.class, listener); + ynlRegistrations.put(id, new AbstractObjectRegistration<>(listener) { + @Override + protected void removeRegistration() { + reg.close(); + } + }); return RpcResultBuilder.success(new SubscribeYnlOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> unregisterBoundConstant( + private ListenableFuture> unregisterBoundConstant( final UnregisterBoundConstantInput input) { LOG.info("In unregisterBoundConstant - {}", input); - final DOMRpcImplementationRegistration rpcRegistration = - routedRegistrations.remove(input.getContext()); - + final var rpcRegistration = routedRegistrations.remove(input.getContext()); if (rpcRegistration == null) { return RpcResultBuilder.failed() .withError(ErrorType.RPC, ErrorTag.DATA_MISSING, @@ -284,8 +335,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(new UnregisterBoundConstantOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> registerSingletonConstant( + private ListenableFuture> registerSingletonConstant( final RegisterSingletonConstantInput input) { LOG.info("In registerSingletonConstant - input: {}", input); @@ -301,14 +351,12 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(new RegisterSingletonConstantOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> registerDefaultConstant( + private ListenableFuture> registerDefaultConstant( final RegisterDefaultConstantInput input) { return null; } - @Override - public ListenableFuture> unregisterConstant( + private ListenableFuture> unregisterConstant( final UnregisterConstantInput input) { LOG.info("In unregisterConstant"); @@ -324,8 +372,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(new UnregisterConstantOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> unregisterFlappingSingleton( + private ListenableFuture> unregisterFlappingSingleton( final UnregisterFlappingSingletonInput input) { LOG.info("In unregisterFlappingSingleton"); @@ -342,18 +389,15 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS .buildFuture(); } - @Override - public ListenableFuture> addShardReplica(final AddShardReplicaInput input) { + private ListenableFuture> addShardReplica(final AddShardReplicaInput input) { throw new UnsupportedOperationException(); } - @Override - public ListenableFuture> subscribeDdtl(final SubscribeDdtlInput input) { + private ListenableFuture> subscribeDdtl(final SubscribeDdtlInput input) { throw new UnsupportedOperationException(); } - @Override - public ListenableFuture> registerBoundConstant( + private ListenableFuture> registerBoundConstant( final RegisterBoundConstantInput input) { LOG.info("In registerBoundConstant - input: {}", input); @@ -373,16 +417,14 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS .buildFuture(); } - final DOMRpcImplementationRegistration rpcRegistration = - RoutedGetConstantService.registerNew(bindingNormalizedNodeSerializer, domRpcService, - input.getConstant(), input.getContext()); + final var rpcRegistration = RoutedGetConstantService.registerNew(bindingNormalizedNodeSerializer, domRpcService, + input.getConstant(), input.getContext()); routedRegistrations.put(input.getContext(), rpcRegistration); return RpcResultBuilder.success(new RegisterBoundConstantOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> registerFlappingSingleton( + private ListenableFuture> registerFlappingSingleton( final RegisterFlappingSingletonInput input) { LOG.info("In registerFlappingSingleton"); @@ -397,8 +439,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(new RegisterFlappingSingletonOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> unsubscribeDtcl(final UnsubscribeDtclInput input) { + private ListenableFuture> unsubscribeDtcl(final UnsubscribeDtclInput input) { LOG.info("In unsubscribeDtcl"); if (idIntsListener == null || dtclReg == null) { @@ -436,10 +477,10 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS .buildFuture(); } - final boolean nodesEqual = idIntsListener.checkEqual(readResult.get()); + final boolean nodesEqual = idIntsListener.checkEqual(readResult.orElseThrow()); if (!nodesEqual) { LOG.error("Final read of id-int does not match IdIntsListener's copy. {}", - idIntsListener.diffWithLocalCopy(readResult.get())); + idIntsListener.diffWithLocalCopy(readResult.orElseThrow())); } return RpcResultBuilder.success(new UnsubscribeDtclOutputBuilder().setCopyMatches(nodesEqual).build()) @@ -452,8 +493,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS } } - @Override - public ListenableFuture> unsubscribeYnl(final UnsubscribeYnlInput input) { + private ListenableFuture> unsubscribeYnl(final UnsubscribeYnlInput input) { LOG.info("In unsubscribeYnl - input: {}", input); if (!ynlRegistrations.containsKey(input.getId())) { @@ -463,16 +503,14 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS .buildFuture(); } - final ListenerRegistration reg = ynlRegistrations.remove(input.getId()); - final UnsubscribeYnlOutput output = reg.getInstance().getOutput(); - - reg.close(); - - return RpcResultBuilder.success().withResult(output).buildFuture(); + try (var reg = ynlRegistrations.remove(input.getId())) { + return RpcResultBuilder.success() + .withResult(reg.getInstance().getOutput()) + .buildFuture(); + } } - @Override - public ListenableFuture> checkPublishNotifications( + private ListenableFuture> checkPublishNotifications( final CheckPublishNotificationsInput input) { LOG.info("In checkPublishNotifications - input: {}", input); @@ -497,8 +535,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(output).buildFuture(); } - @Override - public ListenableFuture> shutdownShardReplica( + private ListenableFuture> shutdownShardReplica( final ShutdownShardReplicaInput input) { LOG.info("In shutdownShardReplica - input: {}", input); @@ -549,8 +586,7 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return rpcResult; } - @Override - public ListenableFuture> registerConstant(final RegisterConstantInput input) { + private ListenableFuture> registerConstant(final RegisterConstantInput input) { LOG.info("In registerConstant - input: {}", input); if (input.getConstant() == null) { @@ -569,15 +605,13 @@ public final class MdsalLowLevelTestProvider implements OdlMdsalLowlevelControlS return RpcResultBuilder.success(new RegisterConstantOutputBuilder().build()).buildFuture(); } - @Override - public ListenableFuture> unregisterDefaultConstant( + private ListenableFuture> unregisterDefaultConstant( final UnregisterDefaultConstantInput input) { throw new UnsupportedOperationException(); } - @Override @SuppressWarnings("checkstyle:IllegalCatch") - public ListenableFuture> unsubscribeDdtl(final UnsubscribeDdtlInput input) { + private ListenableFuture> unsubscribeDdtl(final UnsubscribeDdtlInput input) { throw new UnsupportedOperationException(); } }