Fix checkstyle issues in netvirtsfc
authorSam Hague <shague@redhat.com>
Tue, 6 Oct 2015 19:45:13 +0000 (15:45 -0400)
committerSam Hague <shague@redhat.com>
Tue, 6 Oct 2015 19:45:13 +0000 (15:45 -0400)
Change-Id: I7db518959449958849a74c49a83bde1135ee325f
Signed-off-by: Sam Hague <shague@redhat.com>
openstack/net-virt-sfc/impl/pom.xml
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/AbstractDataTreeListener.java
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/INetvirtSfcDataProcessor.java
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/NetvirtSfcAclListener.java
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/NetvirtSfcClassifierListener.java
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/NetvirtSfcProvider.java
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/openflow13/INetvirtSfcOF13Provider.java
openstack/net-virt-sfc/impl/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/sfc/openflow13/NetvirtSfcOF13Provider.java

index 5e87fb15eed3c3021bfca9d68f8a23507150b30b..194365d755cce9d1ad57ae616f1ce9a32148f183 100644 (file)
@@ -96,6 +96,13 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
 
   <build>
     <plugins>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-checkstyle-plugin</artifactId>
+        <configuration>
+          <propertyExpansion>checkstyle.checker.severity=error</propertyExpansion>
+        </configuration>
+      </plugin>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
index 23a3ed3629763dfc40e0f8d56c7f3247915d323c..dfbe6f0eb790506b9b3e487ca20dc4ae31dba555 100644 (file)
@@ -7,33 +7,29 @@
  */
 
 package org.opendaylight.ovsdb.openstack.netvirt.sfc;
-/**
- * @author Arun Yerra
- */
 
-import com.google.common.base.Preconditions;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import java.util.Collection;
 
+import com.google.common.base.Preconditions;
 import org.opendaylight.controller.md.sal.binding.api.DataObjectModification;
 import org.opendaylight.controller.md.sal.binding.api.DataTreeModification;
 import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeEvent;
 import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.INetvirtSfcOF13Provider;
 import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
-import java.util.Collection;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * AbstractChangeListner implemented basic {@link AsyncDataChangeEvent} processing for
  * netvirt-sfc data change listener.
- *
  */
-public abstract class AbstractDataTreeListener <T extends DataObject> implements INetvirtSfcDataProcessor<T>{
+public abstract class AbstractDataTreeListener<T extends DataObject> implements INetvirtSfcDataProcessor<T> {
     private static final Logger LOG = LoggerFactory.getLogger(AbstractDataTreeListener.class);
     protected INetvirtSfcOF13Provider provider;
     protected final Class<T> clazz;
 
-    public AbstractDataTreeListener (INetvirtSfcOF13Provider provider, Class<T> clazz) {
+    public AbstractDataTreeListener(INetvirtSfcOF13Provider provider, Class<T> clazz) {
         this.provider = Preconditions.checkNotNull(provider, "provider can not be null!");
         this.clazz = Preconditions.checkNotNull(clazz, "Class can not be null!");
     }
@@ -48,21 +44,21 @@ public abstract class AbstractDataTreeListener <T extends DataObject> implements
             final DataObjectModification<T> mod = change.getRootNode();
             LOG.info("Received Data Tree Changed Update of Type={} for Key={}", mod.getModificationType(), key);
             switch (mod.getModificationType()) {
-            case DELETE:
-                remove(key, mod.getDataBefore());
-                break;
-            case SUBTREE_MODIFIED:
-                update(key, mod.getDataBefore(), mod.getDataAfter());
-                break;
-            case WRITE:
-                if (mod.getDataBefore() == null) {
-                    add(key, mod.getDataAfter());
-                } else {
+                case DELETE:
+                    remove(key, mod.getDataBefore());
+                    break;
+                case SUBTREE_MODIFIED:
                     update(key, mod.getDataBefore(), mod.getDataAfter());
-                }
-                break;
-            default:
-                throw new IllegalArgumentException("Unhandled modification type " + mod.getModificationType());
+                    break;
+                case WRITE:
+                    if (mod.getDataBefore() == null) {
+                        add(key, mod.getDataAfter());
+                    } else {
+                        update(key, mod.getDataBefore(), mod.getDataAfter());
+                    }
+                    break;
+                default:
+                    throw new IllegalArgumentException("Unhandled modification type " + mod.getModificationType());
             }
         }
     }
index adaebd21f5737a4f6e6ade9519e2cb238ee73adb..5225b0309286ed7d7bf56ccf3bc0401651689776 100644 (file)
@@ -5,12 +5,8 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
-/**
- * @author Arun Yerra
- *
- */
+package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
 import org.opendaylight.controller.md.sal.binding.api.DataTreeChangeListener;
 import org.opendaylight.yangtools.yang.binding.DataObject;
@@ -19,10 +15,8 @@ import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 /**
  * netvirt-sfc-dcl processor
  * org.opendaylight.ovsdb.openstack.netvirt.sfc
- *
- * Created: Oct 05, 2015
  */
-public interface INetvirtSfcDataProcessor <D extends DataObject> extends AutoCloseable, DataTreeChangeListener<D> {
+public interface INetvirtSfcDataProcessor<D extends DataObject> extends AutoCloseable, DataTreeChangeListener<D> {
 
     /**
      * Method removes DataObject which is identified by InstanceIdentifier.
@@ -33,8 +27,8 @@ public interface INetvirtSfcDataProcessor <D extends DataObject> extends AutoClo
     void remove(InstanceIdentifier<D> identifier, D del);
 
     /**
-     * Method updates the original DataObject to the update DataObject
-     * Both are identified by same InstanceIdentifier
+     * Method updates the original DataObject to the update DataObject.
+     * Both are identified by same InstanceIdentifier.
      *
      * @param identifier - the whole path to DataObject
      * @param original - original DataObject (for update)
index 73e6b1b42921c7483f5e6f330eea98fd38932e49..302be54ae402e7d8f80144f16436ee66d7cb0c55 100644 (file)
@@ -5,23 +5,15 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
-/**
- * @author Arun Yerra
- *
- */
+package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
 import com.google.common.base.Preconditions;
-import java.util.concurrent.Callable;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.DataTreeIdentifier;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.INetvirtSfcOF13Provider;
 import org.opendaylight.ovsdb.utils.mdsal.utils.MdsalUtils;
-import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
-import org.opendaylight.yangtools.concepts.ListenerRegistration;
-
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.AccessLists;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.AccessList;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.AccessListKey;
@@ -31,15 +23,25 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev1410
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.Classifiers;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.Classifier;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.classifier.sffs.Sff;
+import org.opendaylight.yangtools.concepts.ListenerRegistration;
+import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+/**
+ * Data tree listener for AccessList.
+ */
 public class NetvirtSfcAclListener extends AbstractDataTreeListener<AccessList> {
     private static final Logger LOG = LoggerFactory.getLogger(NetvirtSfcAclListener.class);
     private ListenerRegistration<NetvirtSfcAclListener> listenerRegistration;
     private MdsalUtils dbutils;
 
-    public NetvirtSfcAclListener (final INetvirtSfcOF13Provider provider, final DataBroker db) {
+    /**
+     * {@link NetvirtSfcAclListener} constructor.
+     * @param provider OpenFlow 1.3 Provider
+     * @param db MdSal {@link DataBroker}
+     */
+    public NetvirtSfcAclListener(final INetvirtSfcOF13Provider provider, final DataBroker db) {
         super(provider, AccessList.class);
         Preconditions.checkNotNull(db, "DataBroker can not be null!");
 
@@ -48,7 +50,8 @@ public class NetvirtSfcAclListener extends AbstractDataTreeListener<AccessList>
     }
 
     private void registrationListener(final DataBroker db) {
-        final DataTreeIdentifier<AccessList> treeId = new DataTreeIdentifier<>(LogicalDatastoreType.CONFIGURATION, getIetfAclIid());
+        final DataTreeIdentifier<AccessList> treeId =
+                new DataTreeIdentifier<>(LogicalDatastoreType.CONFIGURATION, getIetfAclIid());
         try {
             LOG.info("Registering Data Change Listener for Netvirt AccesList configuration.");
             listenerRegistration = db.registerDataTreeChangeListener(treeId, this);
@@ -79,18 +82,17 @@ public class NetvirtSfcAclListener extends AbstractDataTreeListener<AccessList>
         String aclName = removeDataObj.getAclName();
 
         Classifiers classifiers = dbutils.read(LogicalDatastoreType.CONFIGURATION, getClassifierIid());
-        if(classifiers != null) {
-            for(Classifier classifier : classifiers.getClassifier()) {
-                if(classifier.getAcl().equalsIgnoreCase(aclName)) {
-                    if(classifier.getSffs() != null) {
-                        for(Sff sff : classifier.getSffs().getSff()) {
+        if (classifiers != null) {
+            for (Classifier classifier : classifiers.getClassifier()) {
+                if (classifier.getAcl().equalsIgnoreCase(aclName)) {
+                    if (classifier.getSffs() != null) {
+                        for (Sff sff : classifier.getSffs().getSff()) {
                             provider.removeClassifierRules(sff, removeDataObj);
                         }
                     }
                 }
             }
         }
-        return;
     }
 
     @Override
@@ -105,30 +107,36 @@ public class NetvirtSfcAclListener extends AbstractDataTreeListener<AccessList>
         String aclName = addDataObj.getAclName();
         LOG.debug("Adding accesslist = {}", identifier);
         Classifiers classifiers = dbutils.read(LogicalDatastoreType.CONFIGURATION, getClassifierIid());
-        if(classifiers != null) {
-            for(Classifier classifier : classifiers.getClassifier()) {
-                if(classifier.getAcl().equalsIgnoreCase(aclName)) {
-                    if(classifier.getSffs() != null) {
-                        for(Sff sff : classifier.getSffs().getSff()) {
+        if (classifiers != null) {
+            for (Classifier classifier : classifiers.getClassifier()) {
+                if (classifier.getAcl().equalsIgnoreCase(aclName)) {
+                    if (classifier.getSffs() != null) {
+                        for (Sff sff : classifier.getSffs().getSff()) {
                             provider.addClassifierRules(sff, addDataObj);
                         }
                     }
                 }
             }
         }
-        return;
     }
 
-    private InstanceIdentifier<Classifiers> getClassifierIid () {
+    private InstanceIdentifier<Classifiers> getClassifierIid() {
         return InstanceIdentifier.create(Classifiers.class);
     }
 
-    public InstanceIdentifier<AccessList> getIetfAclIid () {
+    public InstanceIdentifier<AccessList> getIetfAclIid() {
         return InstanceIdentifier.create(AccessLists.class).child(AccessList.class);
     }
 
-    public InstanceIdentifier<AccessListEntry> getIetfAclEntryIid (String aclName, String ruleName) {
-        return InstanceIdentifier.create(AccessLists.class).child(AccessList.class, new AccessListKey(aclName)).
-                child(AccessListEntries.class).child(AccessListEntry.class, new AccessListEntryKey(ruleName));
+    /**
+     * Create an {@link AccessListEntry} {@link InstanceIdentifier}.
+     * @param aclName is the name of the ACL
+     * @param ruleName is the name of the rule
+     * @return the {@link AccessListEntry} {@link InstanceIdentifier}
+     */
+    public InstanceIdentifier<AccessListEntry> getIetfAclEntryIid(String aclName, String ruleName) {
+        return InstanceIdentifier.create(AccessLists.class).child(AccessList.class,
+                new AccessListKey(aclName)).child(AccessListEntries.class).child(AccessListEntry.class,
+                new AccessListEntryKey(ruleName));
     }
 }
index d4108d7d679591169242c65889a1c8a49d751474..df1102f83c5dfe2b371bcd58ccd1e56009295703 100644 (file)
@@ -5,40 +5,42 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
-/**
- * @author Arun Yerra
- *
- */
+package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
 import com.google.common.base.Preconditions;
-import java.util.concurrent.Callable;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.DataTreeIdentifier;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
-import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
-import org.opendaylight.yangtools.concepts.ListenerRegistration;
 import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.INetvirtSfcOF13Provider;
 import org.opendaylight.ovsdb.utils.mdsal.utils.MdsalUtils;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.AccessLists;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.AccessList;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.AccessListKey;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.access.list.access.list.entries.AccessListEntry;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.Classifiers;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.Classifier;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.classifier.Sffs;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.classifier.sffs.Sff;
+import org.opendaylight.yangtools.concepts.ListenerRegistration;
+import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+/**
+ * Data tree listener for Classifier.
+ *
+ * @author Arun Yerra
+ */
 public class NetvirtSfcClassifierListener extends AbstractDataTreeListener<Classifier> {
-
     private static final Logger LOG = LoggerFactory.getLogger(NetvirtSfcClassifierListener.class);
     private MdsalUtils dbutils;
     private ListenerRegistration<NetvirtSfcClassifierListener> listenerRegistration;
 
-    public NetvirtSfcClassifierListener (final INetvirtSfcOF13Provider provider, final DataBroker db) {
+    /**
+     * {@link NetvirtSfcClassifierListener} constructor.
+     * @param provider OpenFlow 1.3 Provider
+     * @param db MdSal {@link DataBroker}
+     */
+    public NetvirtSfcClassifierListener(final INetvirtSfcOF13Provider provider, final DataBroker db) {
         super(provider, Classifier.class);
         Preconditions.checkNotNull(db, "DataBroker can not be null!");
         dbutils = new MdsalUtils(db);
@@ -46,7 +48,8 @@ public class NetvirtSfcClassifierListener extends AbstractDataTreeListener<Class
     }
 
     private void registrationListener(final DataBroker db) {
-        final DataTreeIdentifier<Classifier> treeId = new DataTreeIdentifier<>(LogicalDatastoreType.CONFIGURATION, getClassifierIid());
+        final DataTreeIdentifier<Classifier> treeId =
+                new DataTreeIdentifier<>(LogicalDatastoreType.CONFIGURATION, getClassifierIid());
         try {
             LOG.info("Registering Data Change Listener for Netvirt Classifier configuration.");
             listenerRegistration = db.registerDataTreeChangeListener(treeId, this);
@@ -77,12 +80,13 @@ public class NetvirtSfcClassifierListener extends AbstractDataTreeListener<Class
         String aclName = removeDataObj.getAcl();
         // Read the ACL information from data store and make sure it exists.
         AccessList acl = dbutils.read(LogicalDatastoreType.CONFIGURATION,getIetfAclIid(aclName));
-        if(acl == null) {
+        if (acl == null) {
             LOG.debug("IETF ACL with name ={} is not yet configured. skip this operation", aclName);
             return;
         }
-        if(removeDataObj.getSffs() != null) {
-            for(Sff sff : removeDataObj.getSffs().getSff()) {
+
+        if (removeDataObj.getSffs() != null) {
+            for (Sff sff : removeDataObj.getSffs().getSff()) {
                 // Netvirt classifier binds an ACL with service function forwarder that is identified by SFF name.
                 // SFF validation can be done with SFC Provider APIs, as SFF is configured within SFC project.  
                 // Netvirt SFC provider will validate the SFF using SFC provider APIs.
@@ -105,12 +109,13 @@ public class NetvirtSfcClassifierListener extends AbstractDataTreeListener<Class
         String aclName = addDataObj.getAcl();
         // Read the ACL information from data store and make sure it exists.
         AccessList acl = dbutils.read(LogicalDatastoreType.CONFIGURATION,getIetfAclIid(aclName));
-        if(acl == null) {
+        if (acl == null) {
             LOG.debug("IETF ACL with name ={} is not yet configured. skip this operation", aclName);
             return;
         }
-        if(addDataObj.getSffs() != null) {
-            for(Sff sff : addDataObj.getSffs().getSff()) {
+
+        if (addDataObj.getSffs() != null) {
+            for (Sff sff : addDataObj.getSffs().getSff()) {
                 // Netvirt classifier binds an ACL with service function forwarder that is identified by SFF name.
                 // SFF validation can be done with SFC Provider APIs, as SFF is configured within SFC project.  
                 // Netvirt SFC provider will validate the SFF using SFC provider APIs.
@@ -119,12 +124,11 @@ public class NetvirtSfcClassifierListener extends AbstractDataTreeListener<Class
         }
     }
 
-    public InstanceIdentifier<Classifier> getClassifierIid () {
+    public InstanceIdentifier<Classifier> getClassifierIid() {
         return InstanceIdentifier.create(Classifiers.class).child(Classifier.class);
     }
 
-    private InstanceIdentifier<AccessList> getIetfAclIid (String aclName) {
+    private InstanceIdentifier<AccessList> getIetfAclIid(String aclName) {
         return InstanceIdentifier.create(AccessLists.class).child(AccessList.class, new AccessListKey(aclName));
     }
-
 }
index 91b6c428731e372edf228b985aa89eeb116227aa..144514f9df74da5b8b09c736613b2f3ab28cf374 100644 (file)
@@ -9,19 +9,15 @@
 package org.opendaylight.ovsdb.openstack.netvirt.sfc;
 
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
-import com.google.common.base.Preconditions;
-
-import org.osgi.framework.BundleContext;
-
 import org.opendaylight.controller.sal.binding.api.BindingAwareBroker.ProviderContext;
-import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.NetvirtSfcOF13Provider;
-import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.INetvirtSfcOF13Provider;
 import org.opendaylight.controller.sal.binding.api.BindingAwareProvider;
+import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.INetvirtSfcOF13Provider;
+import org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13.NetvirtSfcOF13Provider;
+import org.osgi.framework.BundleContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public class NetvirtSfcProvider implements BindingAwareProvider, AutoCloseable {
-
     private static final Logger LOG = LoggerFactory.getLogger(NetvirtSfcProvider.class);
     private DataBroker dataBroker = null;
     private BundleContext bundleContext = null;
@@ -30,7 +26,7 @@ public class NetvirtSfcProvider implements BindingAwareProvider, AutoCloseable {
     private NetvirtSfcClassifierListener classfierListener;
     private INetvirtSfcOF13Provider provider;
 
-    public NetvirtSfcProvider (BundleContext bundleContext) {
+    public NetvirtSfcProvider(BundleContext bundleContext) {
         LOG.info("NetvirtProvider: bundleContext: {}", bundleContext);
         this.bundleContext = bundleContext;
     }
@@ -40,9 +36,9 @@ public class NetvirtSfcProvider implements BindingAwareProvider, AutoCloseable {
         LOG.info("NetvirtSfcProvider Session Initiated");
         dataBroker = session.getSALService(DataBroker.class);
 
-        provider = new NetvirtSfcOF13Provider (this.dataBroker);
+        provider = new NetvirtSfcOF13Provider(this.dataBroker);
         aclListener = new NetvirtSfcAclListener(provider, this.dataBroker);
-        classfierListener = new NetvirtSfcClassifierListener (provider, this.dataBroker);
+        classfierListener = new NetvirtSfcClassifierListener(provider, this.dataBroker);
     }
 
     @Override
index ccec011b2c2a9be3bdde22cd766fddb1bc890681..067999ec2d61fdf596728068594ed04ebfe1fd1c 100644 (file)
@@ -1,20 +1,20 @@
 /*
- * Copyright (c) 2013, 2015 Dell, Inc. and others. All rights reserved.
+ * Copyright (c) 2015 Dell, Inc. and others. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-package org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13;
 
-/**
- * @author Arun Yerra
- *
- */
+package org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13;
 
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.AccessList;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.classifier.sffs.Sff;
 
+/**
+ * Open vSwitch OpenFlow 1.3 Networking Provider for Netvirt SFC
+ * @author Arun Yerra
+ */
 public interface INetvirtSfcOF13Provider {
 
     /**
index 3eaa0a1fd64bf8985586ae93beafaffe21e695b5..ca75a5879cf681a5967d9ab3a8fee0f5f7683f5c 100644 (file)
@@ -1,34 +1,29 @@
 /*
- * Copyright (c) 2013, 2015 Dell, Inc. and others. All rights reserved.
+ * Copyright (c) 2015 Dell, Inc. and others. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-package org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13;
 
-/**
- * @author Arun Yerra
- *
- */
+package org.opendaylight.ovsdb.openstack.netvirt.sfc.openflow13;
 
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import com.google.common.base.Preconditions;
-
-import org.opendaylight.controller.sal.binding.api.BindingAwareBroker.ProviderContext;
-import org.opendaylight.controller.sal.binding.api.BindingAwareProvider;
+import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.acl.rev141010.access.lists.AccessList;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.classifier.rev150105.classifiers.classifier.sffs.Sff;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
+/**
+ * Open vSwitch OpenFlow 1.3 Networking Provider for Netvirt SFC
+ * @author Arun Yerra
+ */
 public class NetvirtSfcOF13Provider implements INetvirtSfcOF13Provider{
 
     private final DataBroker dataService;
 
-    public NetvirtSfcOF13Provider (final DataBroker dataBroker) {
+    public NetvirtSfcOF13Provider(final DataBroker dataBroker) {
         this.dataService = Preconditions.checkNotNull(dataBroker, "DataBroker can not be null!");
-   }
+    }
 
     @Override
     public void addClassifierRules(Sff sff, AccessList acl) {