-
Improvement
-
Resolution: Done
-
Medium
-
None
RestconfStreamsSubscriptionService's path currently overlaps with RestconfDataService, i.e. the access is mediated through delegation. As noted in the current FIXME, the fact we are starting the stream through a GET operation is a REST violation. This harks back to the original implementation which dropped here: https://git.opendaylight.org/gerrit/c/controller/+/5094.
Given that the streams can hold a ton of data, we should start them when the first subscriber attaches and stop them when it detaches.
Current organization of the code deals with updating the datastore, which should actually happen when the stream is created, so this change requires cleaning up component wiring.
- is blocked by
-
NETCONF-1099 Restore sal-remote create-notification-stream functionality
- Resolved
- relates to
-
NETCONF-1196 Clean-up RestconfStateStreamsTest class
- Resolved