[JSONRPC-59] Don't use NotificationListener in JSONRPC components Created: 06/Sep/23 Updated: 07/Sep/23 Resolved: 07/Sep/23 |
|
| Status: | Resolved |
| Project: | jsonrpc |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Medium |
| Reporter: | Ruslan Kashapov | Assignee: | Ruslan Kashapov |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
Usage of generated interfaces extending org.opendaylight.yangtools.yang.binding.NotificationListener interface was deprecated for removal via In order to eliminate dependency on deprecated interface the affected logic require update to use org.opendaylight.mdsal.binding.api.NotificationService.Listener instead. Affected components: ./jsonrpc/provider/common/src/main/java/org/opendaylight/jsonrpc/impl/JsonRPCNotificationService.java: public synchronized <T extends DOMNotificationListener> ListenerRegistration<T> registerNotificationListener(
./jsonrpc/provider/common/src/main/java/org/opendaylight/jsonrpc/impl/JsonRPCNotificationService.java: public Registration registerNotificationListeners(final Map<Absolute, DOMNotificationListener> typeToListener) {
./jsonrpc/provider/common/src/test/java/org/opendaylight/jsonrpc/provider/common/RemoteNotificationPublisherTest.java: reg = getDOMNotificationRouter().registerNotificationListener(new DOMNotificationListener() {
./jsonrpc/provider/common/src/test/java/org/opendaylight/jsonrpc/provider/common/JsonRPCNotificationServiceTest.java: regs.add(svc.registerNotificationListener((DOMNotificationListener) notification -> {
./jsonrpc/provider/common/src/test/java/org/opendaylight/jsonrpc/provider/common/JsonRPCNotificationServiceTest.java: svc.registerNotificationListener((DOMNotificationListener) notification -> {
|
| Comments |
| Comment by Ruslan Kashapov [ 07/Sep/23 ] |
|
JSONRPC project does not use MDSAL's Notification service, nothing to update here Resolved as Won't Do |