Uploaded image for project: 'lispflowmapping'
  1. lispflowmapping
  2. LISPMAP-184

Don't use NotificationListener in LISPMAP components

    XMLWordPrintable

Details

    • Task
    • Status: In Review
    • Medium
    • Resolution: Unresolved
    • 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:

      ./lispflowmapping/mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/lisp/MapServer.java:            notificationService.registerNotificationListener(this);
./lispflowmapping/mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/LispMappingService.java:        listenerRegistration = notificationService.registerNotificationListener(this);
./lispflowmapping/integrationtest/src/test/java/org/opendaylight/lispflowmapping/integrationtest/MappingServiceIntegrationTest.java:        ((LispMappingService) lms).getNotificationService().registerNotificationListener(

      Attachments

        Issue Links

          blocks

          Task - A task that needs to be done. MDSAL-497 Remove support for listening on generated NotificationListener interfaces

          • Medium - Has the potential to affect progress.
          • Resolved
          # Subject Branch Project Status CR V
          107937,3 Do not use NotificationListener in LISP components master lispflowmapping Status: NEW +1 +1

          Activity

            People

              matej.sramcik Matej Šramčík
              rkashapov Ruslan Kashapov
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: