Bug 6743: added futures to InterfaceManager and VppNodeManager

[groupbasedpolicy.git] / renderers / vpp / src / main / java / org / opendaylight / groupbasedpolicy / renderer / vpp / iface / VppEndpointLocationProvider.java

diff --git a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/VppEndpointLocationProvider.java b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/VppEndpointLocationProvider.java
index e897ff351f1b06b0a48629a9b3a2cc47f0cabcab..93f770e8ac97eeefda6546e81fee9c354e88465d 100644 (file)
--- a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/VppEndpointLocationProvider.java
+++ b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/VppEndpointLocationProvider.java
@@ -11,7 +11,10 @@ package org.opendaylight.groupbasedpolicy.renderer.vpp.iface;
 import static com.google.common.base.Preconditions.checkNotNull;
 
 import javax.annotation.Nonnull;
-
+import com.google.common.base.Function;
+import com.google.common.util.concurrent.FutureCallback;
+import com.google.common.util.concurrent.Futures;
+import com.google.common.util.concurrent.ListenableFuture;
 import org.opendaylight.controller.md.sal.binding.api.BindingTransactionChain;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
@@ -37,11 +40,6 @@ import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.google.common.base.Function;
-import com.google.common.util.concurrent.FutureCallback;
-import com.google.common.util.concurrent.Futures;
-import com.google.common.util.concurrent.ListenableFuture;
-
 public class VppEndpointLocationProvider implements AutoCloseable {
 
     private static final Logger LOG = LoggerFactory.getLogger(VppEndpointLocationProvider.class);
@@ -74,21 +72,18 @@ public class VppEndpointLocationProvider implements AutoCloseable {
         });
     }
 
-    public ListenableFuture<Void> createLocationForVppEndpoint(VppEndpoint vppEndpoint) {
-        ProviderAddressEndpointLocation providerAddressEndpointLocation = createProviderAddressEndpointLocation(vppEndpoint);
-        WriteTransaction wTx = txChain.newWriteOnlyTransaction();
+    ListenableFuture<Void> createLocationForVppEndpoint(final VppEndpoint vppEndpoint) {
+        final ProviderAddressEndpointLocation providerAddressEndpointLocation =
+                createProviderAddressEndpointLocation(vppEndpoint);
+        final WriteTransaction wTx = txChain.newWriteOnlyTransaction();
         wTx.put(LogicalDatastoreType.CONFIGURATION, IidFactory.providerAddressEndpointLocationIid(
                 VPP_ENDPOINT_LOCATION_PROVIDER, providerAddressEndpointLocation.getKey()),
                 providerAddressEndpointLocation);
         LOG.debug("Creating location for {}", providerAddressEndpointLocation.getKey());
-        return Futures.transform(wTx.submit(), new Function<Void, Void>() {
-
-            @Override
-            public Void apply(Void input) {
-                LOG.debug("{} provided location: {}", VPP_ENDPOINT_LOCATION_PROVIDER.getValue(),
-                        providerAddressEndpointLocation);
-                return null;
-            }
+        return Futures.transform(wTx.submit(), (Function<Void, Void>) input -> {
+            LOG.debug("{} provided location: {}", VPP_ENDPOINT_LOCATION_PROVIDER.getValue(),
+                    providerAddressEndpointLocation);
+            return null;
         });
     }
 
@@ -106,19 +101,16 @@ public class VppEndpointLocationProvider implements AutoCloseable {
             .build();
     }
 
-    public ListenableFuture<Void> deleteLocationForVppEndpoint(VppEndpoint vppEndpoint) {
-        ProviderAddressEndpointLocationKey provAddrEpLocKey = createProviderAddressEndpointLocationKey(vppEndpoint);
-        WriteTransaction wTx = txChain.newWriteOnlyTransaction();
+    ListenableFuture<Void> deleteLocationForVppEndpoint(final VppEndpoint vppEndpoint) {
+        final ProviderAddressEndpointLocationKey provAddrEpLocKey =
+                createProviderAddressEndpointLocationKey(vppEndpoint);
+        final WriteTransaction wTx = txChain.newWriteOnlyTransaction();
         wTx.delete(LogicalDatastoreType.CONFIGURATION,
                 IidFactory.providerAddressEndpointLocationIid(VPP_ENDPOINT_LOCATION_PROVIDER, provAddrEpLocKey));
         LOG.debug("Deleting location for {}", provAddrEpLocKey);
-        return Futures.transform(wTx.submit(), new Function<Void, Void>() {
-
-            @Override
-            public Void apply(Void input) {
-                LOG.debug("{} removed location: {}", VPP_ENDPOINT_LOCATION_PROVIDER.getValue(), provAddrEpLocKey);
-                return null;
-            }
+        return Futures.transform(wTx.submit(), (Function<Void, Void>) input -> {
+            LOG.debug("{} removed location: {}", VPP_ENDPOINT_LOCATION_PROVIDER.getValue(), provAddrEpLocKey);
+            return null;
         });
     }