Uploaded image for project: 'jsonrpc'
  1. jsonrpc
  2. JSONRPC-59

Don't use NotificationListener in JSONRPC components

    XMLWordPrintable

Details

    • Task
    • Status: Resolved
    • Medium
    • Resolution: Won't Do
    • None
    • None
    • None
    • None

    Description

      Usage of generated interfaces extending org.opendaylight.yangtools.yang.binding.NotificationListener interface was deprecated for removal via MDSAL-496.

      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 -> {
      

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              rkashapov Ruslan Kashapov
              rkashapov Ruslan Kashapov
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: