Bug 4948: Catch RuntimeException from ensureParentByMerge 59/33659/6
authorBalaji Varadaraju <bvaradar@brocade.com>
Wed, 27 Jan 2016 20:17:49 +0000 (14:17 -0600)
committerRyan Goulding <ryandgoulding@gmail.com>
Tue, 2 Feb 2016 20:58:43 +0000 (15:58 -0500)
commite26d4960e9ba3b0dc4a2dc05e3ed1c44cafdaf39
tree98f8d4b510d246fc47ea2c25d55838573bca9075
parent5caa8d423874ac4447b60704f56c44b42b13164e
Bug 4948: Catch RuntimeException from ensureParentByMerge

Made the calls to PUT and POST more robust. Caught the RuntimeException from
 the merge call in the EnsureParentsByMerge. If this fails then started
a new transaction and continue with the original PUT. This ensures when a device
is unable to handle the empty list/container merge it will still succeed with the
original request. Similar change has been made for POST.
This bug was fixed in stable Lithium. I am pushing the same fix on Beryllium.

Added FIXME statements to figure out and catch specific Runtime exceptions
thrown by Netconf instead of RuntimeException.

Change-Id: Id18a97857e46f44014dbd61cbfa86285457a2905
Signed-off-by: Balaji Varadaraju <bvaradar@brocade.com>
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
opendaylight/restconf/sal-rest-connector/src/main/java/org/opendaylight/netconf/sal/restconf/impl/BrokerFacade.java