From Hugo Trippaers <hugo@...>:

Hugo Trippaers has posted comments on this change.

Change subject: Implemented the deleteBridgeDomain method in ConfigurationService
......................................................................


Patch Set 1: Code-Review+1

(2 comments)

You beat me to it, was just working on the same thing ;-)

....................................................
File ovsdb/src/main/java/org/opendaylight/ovsdb/plugin/ConfigurationService.java
Line 606: if (tr.get(i).getError() != null && tr.get(i).getError().trim().length() > 0) {
With openvswitch it is possible that a transaction returns an array with null elements. As there is only one transaction here it is not a big risk. But for safety we should check if tr.get(i) != null


Line 877: status = this.deleteBridgeDomain(Node.fromString(nodeName), bridgeName);
Node.fromString can return null. deleteBridgeDomain doesn't check this resulting in an NPE. Should we check for this even though this is a debug function?