Uploaded image for project: 'netconf'
  1. netconf
  2. NETCONF-1104

Turn ListenersBroker into a component

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved
    • Highest
    • Resolution: Done
    • None
    • 7.0.0
    • restconf-nb

    Description

      ListenersBroker hosts node-local state about what notification streams have been defined. At the end of the day, it it has three distinct parts:

      • dataChangeListeners, backed by DOMDataTreeChangeService
      • notificationListeners, backed by DOMNotificationService
      • deviceNotificationListeners, backed by NETCONF mounted devices

      This is a ClassLoader singleton, accessed via static ListenersBroker.getInstance() – which is completely wrong, as it should be a a normal singleton managing the contents of ietf-restconf-monitoring streams' content, with the three parts managed separately as:

      • SalRemoteImpl, which handles dataChangeListeners and notificationListeners
      • OdlDeviceNotificationImpl, which handles deviceNotificationListeners

      This deals with the first part of the problem: getting rid of ListenersBroker.getInstance() and properly injecting the appropriate service into all components accessing ListenersBroker instances.

      Attachments

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

        Activity

          People

            ojo Oleksandr Zharov
            rovarga Robert Varga
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: