Transition ListenerAdapter to ClusteredDOMDataTreeListener 64/63264/14
authorJosh <jhershbe@redhat.com>
Tue, 19 Sep 2017 07:42:39 +0000 (10:42 +0300)
committerJosh <jhershbe@redhat.com>
Sun, 15 Oct 2017 10:50:16 +0000 (13:50 +0300)
commit552a5e7e85278c20319693d269c177d9c5cc756e
tree6323df2af162c0576dab61eab5ce20ce562c06ef
parent01d82cd895e8c81011c613c4c6af9dcc362110ea
Transition ListenerAdapter to ClusteredDOMDataTreeListener

Solves this: https://bugs.opendaylight.org/show_bug.cgi?id=9147
This is in general an issue with ListenerAdapter which does not
currently work in a clustered deployment.

ListenerAdapterTest needed to be adjusted for two reasons:
1. There was a bug in ListenerAdapter where if leafNodesOnly
was set the leafNodesOnly logic was not applied in delete
notifications. The new ListenerAdapter code fixes that so
ListenerAdapterTest needed to be adjusted.
2. In DomDataTreeEtc, merge operations cause the key node
to count as "WRITE"-en for notifications even if it was not
changed. ListenerAdapterTest was adjusted accordingly. Bug
for this here: https://bugs.opendaylight.org/show_bug.cgi?id=9179

Change-Id: I3c40e02551d8f04354489e5ffb8c66f111bb3b96
Signed-off-by: Josh <jhershbe@redhat.com>
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/restconf/impl/BrokerFacade.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/streams/listeners/ListenerAdapter.java
restconf/restconf-nb-bierman02/src/test/java/org/opendaylight/controller/sal/restconf/impl/test/BrokerFacadeTest.java
restconf/restconf-nb-bierman02/src/test/java/org/opendaylight/controller/sal/restconf/impl/test/RestconfImplNotificationSubscribingTest.java
restconf/restconf-nb-bierman02/src/test/java/org/opendaylight/netconf/sal/streams/listeners/ListenerAdapterTest.java
restconf/restconf-nb-bierman02/src/test/resources/listener-adapter-test/notif-leaves-del.json
restconf/restconf-nb-bierman02/src/test/resources/listener-adapter-test/notif-leaves-update.json
restconf/restconf-nb-bierman02/src/test/resources/listener-adapter-test/notif-update.json