https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/NETCONF-1150 netconf <p>Usage of generated interfaces extending org.opendaylight.yangtools.yang.binding.<b>NotificationListener</b> interface was deprecated for removal via <a href="https://jira.opendaylight.org/browse/MDSAL-496" title="Deprecate generated notification listener interface" class="issue-link" data-issue-key="MDSAL-496"><del>MDSAL-496</del></a>.</p> <p>In order to eliminate dependency on deprecated interface the affected logic require update to use org.opendaylight.mdsal.binding.api.NotificationService.<b>Listener</b> instead.</p> <p>Affected components:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>./netconf/plugins/netconf-server-mdsal/src/main/java/org/opendaylight/netconf/server/mdsal/notifications/CreateSubscription.java:        subscriptions.add(notifications.registerNotificationListener(streamNameType, ./netconf/plugins/netconf-server-mdsal/src/main/java/org/opendaylight/netconf/server/mdsal/notifications/NetconfNotificationManager.java:    public synchronized Registration registerNotificationListener(final StreamNameType stream, ./netconf/plugins/netconf-server-mdsal/src/test/java/org/opendaylight/netconf/server/mdsal/notifications/NetconfNotificationManagerTest.java:        try (var reg = netconfNotificationManager.registerNotificationListener( ./netconf/plugins/netconf-server-mdsal/src/test/java/org/opendaylight/netconf/server/mdsal/notifications/NetconfNotificationManagerTest.java:        netconfNotificationManager.registerNotificationListener( ./netconf/plugins/netconf-server-mdsal/src/test/java/org/opendaylight/netconf/server/mdsal/notifications/NetconfNotificationManagerTest.java:        netconfNotificationManager.registerNotificationListener(testStream.getName(), listener); ./netconf/plugins/netconf-server-mdsal/src/test/java/org/opendaylight/netconf/server/mdsal/notifications/NetconfNotificationManagerTest.java:                .registerNotificationListener(NetconfNotificationManager.BASE_NETCONF_STREAM.getName(), listener); ./netconf/plugins/netconf-server-mdsal/src/test/java/org/opendaylight/netconf/server/mdsal/notifications/CreateSubscriptionTest.java:                .registerNotificationListener(any(StreamNameType.class), any(NetconfNotificationListener.class)); ./netconf/plugins/netconf-client-mdsal/src/main/java/org/opendaylight/netconf/client/mdsal/spi/NetconfDeviceNotificationService.java:    public synchronized &lt;T extends DOMNotificationListener&gt; ListenerRegistration&lt;T&gt; registerNotificationListener( ./netconf/plugins/netconf-client-mdsal/src/main/java/org/opendaylight/netconf/client/mdsal/spi/NetconfDeviceNotificationService.java:    public synchronized &lt;T extends DOMNotificationListener&gt; ListenerRegistration&lt;T&gt; registerNotificationListener( ./netconf/plugins/netconf-client-mdsal/src/main/java/org/opendaylight/netconf/client/mdsal/spi/NetconfDeviceNotificationService.java:        return registerNotificationListener(listener, Lists.newArrayList(types)); ./netconf/plugins/netconf-client-mdsal/src/main/java/org/opendaylight/netconf/client/mdsal/spi/NetconfDeviceNotificationService.java:    public synchronized Registration registerNotificationListeners( ./netconf/plugins/netconf-client-mdsal/src/test/java/org/opendaylight/netconf/client/mdsal/spi/NetconfDeviceNotificationServiceTest.java:        service.registerNotificationListener(listener1, path1); ./netconf/plugins/netconf-client-mdsal/src/test/java/org/opendaylight/netconf/client/mdsal/spi/NetconfDeviceNotificationServiceTest.java:        registration = service.registerNotificationListener(listener2, path2); ./netconf/netconf/tools/netconf-test-perf/src/main/java/org/opendaylight/netconf/test/perf/MountedDeviceListener.java:        listeners.put(path, notificationService.registerNotificationListener( ./netconf/protocol/netconf-server/src/main/java/org/opendaylight/netconf/server/api/notifications/NetconfNotificationRegistry.java:    @NonNull Registration registerNotificationListener(@NonNull StreamNameType stream, ./netconf/restconf/sal-restconf-broker/src/main/java/org/opendaylight/netconf/sal/restconf/broker/impl/DataBrokerServiceImpl.java:        final ListenerRegistration&lt;?&gt; reg = restConfListenableEventStreamContext.registerNotificationListener(remoteDataChangeNotificationListener); ./netconf/restconf/sal-restconf-broker/src/main/java/org/opendaylight/netconf/sal/restconf/broker/impl/NotificationServiceImpl.java:    public &lt;T extends Notification&gt; ListenerRegistration&lt;NotificationListener&lt;T&gt;&gt; registerNotificationListener(Class&lt;T&gt; notificationType, NotificationListener&lt;T&gt; listener) { ./netconf/restconf/sal-restconf-broker/src/main/java/org/opendaylight/netconf/sal/restconf/broker/impl/NotificationServiceImpl.java:                .registerNotificationListener(remoteNotificationListener); ./netconf/restconf/sal-restconf-broker/src/main/java/org/opendaylight/netconf/sal/restconf/broker/impl/NotificationServiceImpl.java:    public ListenerRegistration&lt;org.opendaylight.yangtools.yang.binding.NotificationListener&gt; registerNotificationListener(org.opendaylight.yangtools.yang.binding.NotificationListener listener) { ./netconf/restconf/sal-restconf-broker/src/main/java/org/opendaylight/netconf/sal/restconf/broker/impl/NotificationServiceImpl.java:        return restconfClientContext.getEventStreamContext(desiredEventStream.get(desiredEventStream.get(notificationStreamName))).registerNotificationListener(listener); ./netconf/restconf/restconf-nb/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataServiceImpl.java:            : listenersBroker.registerNotificationListener(Absolute.of(notifName), streamName, outputType); ./netconf/restconf/restconf-nb/src/main/java/org/opendaylight/restconf/nb/rfc8040/streams/listeners/DeviceNotificationListenerAdaptor.java:            setRegistration(notificationService.registerNotificationListener(this, paths)); ./netconf/restconf/restconf-nb/src/main/java/org/opendaylight/restconf/nb/rfc8040/streams/listeners/NotificationListenerAdapter.java:            setRegistration(notificationService.registerNotificationListener(this, getSchemaPath())); ./netconf/restconf/restconf-nb/src/main/java/org/opendaylight/restconf/nb/rfc8040/streams/listeners/ListenersBroker.java:    public NotificationListenerAdapter registerNotificationListener(final Absolute schemaPath, ./netconf/restconf/restconf-nb/src/test/java/org/opendaylight/restconf/nb/rfc8040/streams/listeners/JsonNotificationListenerTest.java:        final NotificationListenerAdapter notifiAdapter = ListenersBroker.getInstance().registerNotificationListener( ./netconf/restconf/restconf-nb/src/test/java/org/opendaylight/restconf/nb/rfc8040/streams/listeners/XmlNotificationListenerTest.java:        final NotificationListenerAdapter notifiAdapter = ListenersBroker.getInstance().registerNotificationListener(</pre> </div></div> NETCONF-1150

Don't use NotificationListener in NETCONF components

Task Medium Resolved Won't Do Ruslan Kashapov Ruslan Kashapov Wed, 6 Sep 2023 10:22:25 +0000 Thu, 7 Sep 2023 09:45:34 +0000 Thu, 7 Sep 2023 09:45:34 +0000 0 1 <p>NETCONF uses DOMNotificationListener only, no dependency on MDSAL deprecated interface<br/> except sal-restconf-broker which is excluded from build and targeted for re-implementation.<br/> Nothing to do here. resolving as  Won't Do</p> Blocks MDSAL-497 Development Rank 0|i046gn: