Fix Jdk8 compatibility 93/17293/4
authorRobert Varga <rovarga@cisco.com>
Sat, 28 Mar 2015 17:48:33 +0000 (18:48 +0100)
committerRobert Varga <rovarga@cisco.com>
Tue, 31 Mar 2015 14:39:08 +0000 (16:39 +0200)
JDK8 has stricter javadoc rules, make sure we do not fail on them.

Change-Id: I13ae2d0cc750faf66dc4f0c41a7982f9e9aae4cb
Signed-off-by: Robert Varga <rovarga@cisco.com>
56 files changed:
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/FlowNodeReconciliation.java
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/ForwardingRulesCommiter.java
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/ForwardingRulesManager.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/StatNodeRegistration.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/StatPermCollector.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/StatRpcMsgManager.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/StatisticsManager.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/impl/StatAbstractListenCommit.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/impl/StatAbstractNotifyCommit.java
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/impl/StatisticsManagerImpl.java
extension/openflowjava-extension-nicira/src/main/java/org/opendaylight/openflowjava/nx/NiciraExtensionsRegistrator.java
extension/openflowplugin-extension-api/src/main/java/org/opendaylight/openflowplugin/extension/api/GroupingLooseResolver.java
extension/openflowplugin-extension-api/src/main/java/org/opendaylight/openflowplugin/extension/api/GroupingResolver.java
extension/openflowplugin-extension-api/src/main/java/org/opendaylight/openflowplugin/extension/api/core/extension/ExtensionConverterProvider.java
extension/openflowplugin-extension-api/src/main/java/org/opendaylight/openflowplugin/extension/api/path/ActionPath.java
legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/ToSalConversionsUtils.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/OpenFlowPluginProvider.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/connection/ConnectionContext.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/connection/ConnectionManager.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/DeviceContext.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/DeviceManager.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/DeviceState.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/RequestContext.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/RequestFutureContext.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/Xid.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/XidGenerator.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/exception/RequestQuotaExceededException.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/handlers/DeviceConnectedHandler.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/handlers/DeviceContextReadyHandler.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/handlers/DeviceSynchronizedHandler.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/handlers/MessageHandler.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/md/core/ConnectionConductor.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/md/queue/QueueKeeper.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/md/queue/QueueProcessor.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/md/util/OpenflowVersion.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/md/util/PollableQueuesPriorityZipper.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/md/util/PollableQueuesZipper.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/rpc/RpcContext.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/rpc/RpcManager.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/statistics/StatisticsContext.java
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/statistics/StatisticsManager.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenflowPluginProviderImpl.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/services/RequestInputUtils.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/services/RpcResultConvertor.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/HandshakeManagerImpl.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/OFRpcTaskFactory.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/OFRpcTaskUtil.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/ActionUtil.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/IpConversionUtil.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/queue/QueueKeeperHarvester.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/queue/QueueProcessorLightImpl.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/queue/WrapperQueueImpl.java
samples/learning-switch/src/main/java/org/opendaylight/openflowplugin/learningswitch/Activator.java
samples/learning-switch/src/main/java/org/opendaylight/openflowplugin/learningswitch/InstanceIdentifierUtils.java
samples/learning-switch/src/main/java/org/opendaylight/openflowplugin/learningswitch/LearningSwitchManager.java
test-provider/src/main/java/org/opendaylight/openflowplugin/test/OpenflowpluginTestCommandProvider.java

index ff46b45f2bb6601ea32bdf656dc41e4e9df6636f..4b762c1a7ffc58706612e014e01b50ab9c208e8c 100644 (file)
@@ -49,8 +49,8 @@ public interface FlowNodeReconciliation extends DataChangeListener, AutoCloseabl
     void flowNodeConnected(InstanceIdentifier<FlowCapableNode> connectedNode);
 
     /**
-     * Method contains functionality for registered Node {@FlowCapableNode} removing
-     * from {@Link ForwardingRulesManager}
+     * Method contains functionality for registered Node {@link FlowCapableNode} removing
+     * from {@link ForwardingRulesManager}
      *
      * @param disconnectedNode - {@link org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier} to removed Node
      */
index df7f6f4affe1e859eaec7b790e8aa20957205f6c..34b6f40e3b74fd98d5b650fc5bc3631da44c7293 100644 (file)
@@ -32,9 +32,9 @@ public interface ForwardingRulesCommiter <D extends DataObject> extends AutoClos
      * Method removes DataObject which is identified by InstanceIdentifier
      * from device.
      *
-     * @param InstanceIdentifier identifier - the whole path to DataObject
-     * @param DataObject remove - DataObject for removing
-     * @param InstanceIdentifier<FlowCapableNode> parent Node InstanceIdentifier
+     * @param identifier - the whole path to DataObject
+     * @param del - DataObject for removing
+     * @param nodeIdent Node InstanceIdentifier
      */
     void remove(InstanceIdentifier<D> identifier, D del,
             InstanceIdentifier<FlowCapableNode> nodeIdent);
@@ -43,9 +43,10 @@ public interface ForwardingRulesCommiter <D extends DataObject> extends AutoClos
      * Method updates the original DataObject to the update DataObject
      * in device. Both are identified by same InstanceIdentifier
      *
-     * @param InstanceIdentifier identifier - the whole path to DataObject
-     * @param DataObject original - original DataObject (for update)
-     * @param DataObject update - changed DataObject (contain updates)
+     * @param identifier - the whole path to DataObject
+     * @param original - original DataObject (for update)
+     * @param update - changed DataObject (contain updates)
+     * @param nodeIdent Node InstanceIdentifier
      */
     void update(InstanceIdentifier<D> identifier, D original, D update,
             InstanceIdentifier<FlowCapableNode> nodeIdent);
@@ -54,8 +55,9 @@ public interface ForwardingRulesCommiter <D extends DataObject> extends AutoClos
      * Method adds the DataObject which is identified by InstanceIdentifier
      * to device.
      *
-     * @param InstanceIdentifier identifier - the whole path to new DataObject
-     * @param DataObject add - new DataObject
+     * @param identifier - the whole path to new DataObject
+     * @param add - new DataObject
+     * @param nodeIdent Node InstanceIdentifier
      */
     void add(InstanceIdentifier<D> identifier, D add,
             InstanceIdentifier<FlowCapableNode> nodeIdent);
index c6b63d1d4886b039a38915c7556edae2ed8fb129..4c695f2837143adee9217c451faaaa99b940706a 100644 (file)
@@ -39,8 +39,8 @@ public interface ForwardingRulesManager extends AutoCloseable {
      * Method returns information :
      * "is Node with send InstanceIdentifier connected"?
      *
-     * @param InstanceIdentifier<FlowCapableNode> ident - the key of the node
-     * @return boolean - is device connected
+     * @param ident - the key of the node
+     * @return boolean - true if device is connected
      */
     public boolean isNodeActive(InstanceIdentifier<FlowCapableNode> ident);
 
@@ -49,7 +49,7 @@ public interface ForwardingRulesManager extends AutoCloseable {
      * ActiveNodeHolder prevent unnecessary Operational/DS read for identify
      * pre-configure and serious Configure/DS transactions.
      *
-     * @param InstanceIdentifier<FlowCapableNode> ident - the key of the node
+     * @param ident - the key of the node
      */
     public void registrateNewNode(InstanceIdentifier<FlowCapableNode> ident);
 
@@ -58,7 +58,7 @@ public interface ForwardingRulesManager extends AutoCloseable {
      * Holder. And all next flows or groups or meters will stay in Config/DS
      * only.
      *
-     * @param InstanceIdentifier<FlowCapableNode> ident - the key of the node
+     * @param ident - the key of the node
      */
     public void unregistrateNode(InstanceIdentifier<FlowCapableNode> ident);
 
@@ -100,19 +100,19 @@ public interface ForwardingRulesManager extends AutoCloseable {
 
     /**
      * Content definition method and prevent code duplicity in Reconcil
-     * @return ForwardingRulesCommiter<Flow>
+     * @return ForwardingRulesCommiter&lt;Flow&gt;
      */
     public ForwardingRulesCommiter<Flow> getFlowCommiter();
 
     /**
      * Content definition method and prevent code duplicity in Reconcil
-     * @return ForwardingRulesCommiter<Group>
+     * @return ForwardingRulesCommiter&lt;Group&gt;
      */
     public ForwardingRulesCommiter<Group> getGroupCommiter();
 
     /**
      * Content definition method and prevent code duplicity
-     * @return ForwardingRulesCommiter<Meter>
+     * @return ForwardingRulesCommiter&lt;Meter&gt;
      */
     public ForwardingRulesCommiter<Meter> getMeterCommiter();
 
index ffd8132cf381282a7fb7f51bda48799a2f8eaf06..178befea06e3124014fee016a0e8b108ce0b5d6b 100644 (file)
@@ -34,9 +34,9 @@ public interface StatNodeRegistration extends OpendaylightInventoryListener, Aut
      * as a prevention to use a validation check to the Operational/DS for identify
      * connected {@link org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNode}.
      *
-     * @param InstanceIdentifier<SwitchFeatures> keyIdent
-     * @param FlowCapableNode data
-     * @param InstanceIdentifier<Node> nodeIdent
+     * @param keyIdent
+     * @param data
+     * @param nodeIdent
      */
     void connectFlowCapableNode(InstanceIdentifier<SwitchFeatures> keyIdent,
             SwitchFeatures data, InstanceIdentifier<Node> nodeIdent);
@@ -44,7 +44,7 @@ public interface StatNodeRegistration extends OpendaylightInventoryListener, Aut
     /**
      * Method cut {@link Node} registration for {@link StatPermCollector}
      *
-     * @param InstanceIdentifier<Node> keyIdent
+     * @param keyIdent
      */
     void disconnectFlowCapableNode(InstanceIdentifier<Node> keyIdent);
 }
index a57dbcf6e2d0d267f754f32fff631a812c8f958a..1f4432b5ec769d3b3aa3958cc7fc6353e7f90711 100644 (file)
@@ -73,7 +73,7 @@ public interface StatPermCollector extends Runnable, AutoCloseable {
     /**
      * Add new connected node for permanent statistics collecting process
      *
-     * @param flowNode
+     * @param nodeIdent
      * @param statTypes
      * @param nrOfSwitchTables
      * @return true/false if the {@link Node} added successful
@@ -84,16 +84,16 @@ public interface StatPermCollector extends Runnable, AutoCloseable {
     /**
      * All disconnected Nodes need be removed from stat list Nodes
      *
-     * @param flowNode
+     * @param nodeIdent
      * @return true/false if the {@link Node} removed successful
      */
     boolean disconnectedNodeUnregistration(InstanceIdentifier<Node> nodeIdent);
 
     /**
      * Method add new feature {@link StatCapabTypes} to Node identified by
-     * nodeIdent -> InstanceIdentifier<Node>
+     * nodeIdent -&gt; InstanceIdentifier&lt;Node&gt;
      *
-     * @param flowNode
+     * @param nodeIdent
      * @return true/false if the {@link StatCapabTypes} add successful
      */
     boolean registerAdditionalNodeFeature(InstanceIdentifier<Node> nodeIdent, StatCapabTypes statCapab);
@@ -103,7 +103,7 @@ public interface StatPermCollector extends Runnable, AutoCloseable {
      * valid node registration in its internal {@link Node} map.
      * Otherwise return false.
      *
-     * @param flowNode
+     * @param nodeIdent
      * @return
      */
     boolean isProvidedFlowNodeActive(InstanceIdentifier<Node> nodeIdent);
index 984650c72f7e18aacf144837a75569a94a3f101b..d5532a0e337449855b201ffe09b90eb55fab30b2 100644 (file)
@@ -46,9 +46,9 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Transaction container is definition for Multipart transaction
      * join container for all Multipart msg with same TransactionId
      * Input {@link DataObject} is a possible light-weight DataObject
-     * which is used for identification (e.g. Flow-> Priority,Match,Cookie,FlowId)
+     * which is used for identification (e.g. Flow -&gt; Priority,Match,Cookie,FlowId)
      *
-     * @param <T> extends TransactionAware -
+     * @param <T>
      */
     interface TransactionCacheContainer<T extends TransactionAware> {
 
@@ -67,7 +67,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method is used for check a transaction registration
      * for multipart cache holder
      *
-     * @param TransactionId id
+     * @param id
      * @return true if the transaction has been correctly registered
      */
     Future<Boolean> isExpectedStatistics(TransactionId id, NodeId nodeId);
@@ -95,7 +95,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * and build all to TransactionCacheContainer Object to return. This process clean
      * all instances in Cache.
      *
-     * @param TransactionId id
+     * @param id
      * @return TransactionCacheContainer
      */
     Future<Optional<TransactionCacheContainer<?>>> getTransactionCacheContainer(TransactionId id, NodeId nodeId);
@@ -104,7 +104,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightGroupStatisticsService.getAllGroupStatistics
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllGroupsStat(NodeRef nodeRef);
 
@@ -112,7 +112,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightGroupStatisticsService.getGroupDescription
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllGroupsConfStats(NodeRef nodeRef);
 
@@ -120,7 +120,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightMeterStatisticsService.getGroupFeatures
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     void getGroupFeaturesStat(NodeRef nodeRef);
 
@@ -128,7 +128,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightMeterStatisticsService.getAllMeterStatistics
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllMetersStat(NodeRef nodeRef);
 
@@ -136,7 +136,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightMeterStatisticsService.getAllMeterConfigStatistics
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllMeterConfigStat(NodeRef nodeRef);
 
@@ -144,7 +144,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightMeterStatisticsService.getMeterFeatures
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     void getMeterFeaturesStat(NodeRef nodeRef);
 
@@ -152,7 +152,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightFlowStatisticsService.getAllFlowsStatisticsFromAllFlowTables
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllFlowsStat(NodeRef nodeRef);
 
@@ -160,8 +160,8 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightFlowStatisticsService.getAggregateFlowStatisticsFromFlowTableForAllFlows
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
-     * @param TableId tableId
+     * @param nodeRef
+     * @param tableId
      */
     void getAggregateFlowStat(NodeRef nodeRef, TableId tableId);
 
@@ -169,7 +169,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightPortStatisticsService.getAllNodeConnectorsStatistics
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllPortsStat(NodeRef nodeRef);
 
@@ -177,7 +177,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightFlowTableStatisticsService.getFlowTablesStatistics
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllTablesStat(NodeRef nodeRef);
 
@@ -185,7 +185,7 @@ public interface StatRpcMsgManager extends Runnable, AutoCloseable {
      * Method wraps OpendaylightQueueStatisticsService.getAllQueuesStatisticsFromAllPorts
      * and registers to Transaction Cache
      *
-     * @param NodeRef nodeRef
+     * @param nodeRef
      */
     Future<TransactionId> getAllQueueStat(NodeRef nodeRef);
 
index 5862e717a345b1243697ef7334e7e57ac721fa5c..e1c3fbc1ef57b390e2614b45ed4305bc1e20d136 100644 (file)
@@ -38,7 +38,7 @@ import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
  * StatisticsManager
  * It represent a central point for whole module. Implementation
  * StatisticsManager registers all Operation/DS {@link StatNotifyCommiter} and
- * Config/DS {@StatListeningCommiter}, as well as {@link StatPermCollector}
+ * Config/DS {@link StatListeningCommiter}, as well as {@link StatPermCollector}
  * for statistic collecting and {@link StatRpcMsgManager} as Device RPCs provider.
  * In next, StatisticsManager provides all DS contact Transaction services.
  *
@@ -92,7 +92,7 @@ public interface StatisticsManager extends AutoCloseable, TransactionChainListen
         /**
          * Apply all read / write (put|merge) operation for DataStore
          *
-         * @param {@link ReadWriteTransaction} tx
+         * @param tx {@link ReadWriteTransaction}
          */
         public abstract void applyOperation(ReadWriteTransaction tx);
 
@@ -101,9 +101,8 @@ public interface StatisticsManager extends AutoCloseable, TransactionChainListen
     /**
      * Method starts whole StatisticManager functionality
      *
-     * @param {@link NotificationProviderService} notifService
-     * @param {@link RpcConsumerRegistry} rpcRegistry
-     * @param minReqNetMonitInt
+     * @param notifService
+     * @param rpcRegistry
      */
     void start(final NotificationProviderService notifService,
             final RpcConsumerRegistry rpcRegistry);
@@ -117,7 +116,7 @@ public interface StatisticsManager extends AutoCloseable, TransactionChainListen
     void enqueue(final StatDataStoreOperation inventoryOper);
 
     /**
-     * Method wraps {@link StatisticCollector}.isProvidedFlowNodeActive method
+     * Method wraps {@link StatisticsManager#isProvidedFlowNodeActive(InstanceIdentifier)} method
      * to provide parallel statCollection process for Set of Nodes. So it has to
      * identify correct Node Set by NodeIdentifier
      *
@@ -177,14 +176,14 @@ public interface StatisticsManager extends AutoCloseable, TransactionChainListen
 
     /**
      * Define Method : {@link org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNode}
-     * Operational/DS data change listener -> impl. target -> register FlowCapableNode to Statistic Collecting process
+     * Operational/DS data change listener -&gt; impl. target -&gt; register FlowCapableNode to Statistic Collecting process
      * @return {@link StatNodeRegistration}
      */
     StatNodeRegistration getNodeRegistrator();
 
     /**
-     * Define Method : Flow Config/DS data change listener -> impl. target ->
-     * -> make pair between Config/DS FlowId and Device Flow response Hash
+     * Define Method : Flow Config/DS data change listener -&gt; impl. target -&gt;
+     * -&gt; make pair between Config/DS FlowId and Device Flow response Hash
      * @return
      */
     StatListeningCommiter<Flow, OpendaylightFlowStatisticsListener> getFlowListenComit();
index cb20980b17075525b6c1d638f5d718e338b2b60e..05315108b4718212e4b70050d2826e44771b2e7d 100644 (file)
@@ -81,7 +81,7 @@ public abstract class StatAbstractListenCommit<T extends DataObject, N extends N
     public void onDataChanged(final AsyncDataChangeEvent<InstanceIdentifier<?>, DataObject> changeEvent) {
         Preconditions.checkNotNull(changeEvent,"Async ChangeEvent can not be null!");
         /*
-         * If we have opened read transaction for configuraiton data store,
+         * If we have opened read transaction for configuration data store,
          * we will close and null it.
          *
          * Latest read transaction will be allocated on another read using readLatestConfiguration
index 3ab5dbab42c2921278454f3418ebd928aaabcd00..ad866feb6d96dc12353bf6555638d2f5f8b9329b 100644 (file)
@@ -77,10 +77,10 @@ public abstract class StatAbstractNotifyCommit<N extends NotificationListener> i
     protected abstract N getStatNotificationListener();
 
     /**
-     * PreConfigurationCheck - Node identified by input InstanceIdentifier<Node>
+     * PreConfigurationCheck - Node identified by input InstanceIdentifier&lt;Node&gt;
      * has to be registered in {@link org.opendaylight.openflowplugin.applications.statistics.manager.StatPermCollector}
      *
-     * @param InstanceIdentifier<Node> nodeIdent
+     * @param nodeIdent
      */
     protected boolean preConfigurationCheck(final InstanceIdentifier<Node> nodeIdent) {
         Preconditions.checkNotNull(nodeIdent, "FlowCapableNode ident can not be null!");
@@ -97,8 +97,8 @@ public abstract class StatAbstractNotifyCommit<N extends NotificationListener> i
      * getTransactionCacheContainer with 10sec TimeOut.
      * Method has returned {@link Optional} which could contains a {@link TransactionCacheContainer}
      *
-     * @param TransactionId transId
-     * @param NodeId nodeId
+     * @param transId
+     * @param nodeId
      * @return
      */
     protected Optional<TransactionCacheContainer<?>> getTransactionCacheContainer(final TransactionId transId, final NodeId nodeId) {
@@ -141,8 +141,8 @@ public abstract class StatAbstractNotifyCommit<N extends NotificationListener> i
      * isExpectedStatistics with 10sec TimeOut.
      * Method has checked registration for provided {@link TransactionId} and {@link NodeId}
      *
-     * @param TransactionId transId - Transaction identification
-     * @param NodeId nodeId - Node identification
+     * @param transId - Transaction identification
+     * @param nodeId - Node identification
      * @return boolean
      */
     protected boolean isExpectedStatistics(final TransactionId transId, final NodeId nodeId) {
index 9d9446426e1ae6342d948768ef47da4a56f7b677..347941518d9566deb960194267071299c860247e 100644 (file)
@@ -58,7 +58,7 @@ import com.google.common.util.concurrent.ThreadFactoryBuilder;
 * StatisticsManagerImpl
 * It represent a central point for whole module. Implementation
 * {@link StatisticsManager} registers all Operation/DS {@link StatNotifyCommiter} and
-* Config/DS {@StatListeningCommiter}, as well as {@link StatPermCollector}
+* Config/DS {@link StatListeningCommiter}, as well as {@link StatPermCollector}
 * for statistic collecting and {@link StatRpcMsgManager} as Device RPCs provider.
 * In next, StatisticsManager provides all DS contact Transaction services.
 *
index 52c1cb037136a069cec346a5d7dc0a06cfb5a6b1..53abd822b9c2b67339b699a1b8d060313c302420 100644 (file)
@@ -36,11 +36,10 @@ public class NiciraExtensionsRegistrator implements AutoCloseable {
     private final NiciraExtensionCodecRegistrator registrator;
 
     /**
-     * @param providers cannot be null
+     * @param registrator cannot be null
      */
     public NiciraExtensionsRegistrator(NiciraExtensionCodecRegistrator registrator) {
-        Preconditions.checkNotNull(registrator);
-        this.registrator = registrator;
+        this.registrator = Preconditions.checkNotNull(registrator);
     }
 
     public void registerNiciraExtensions() {
index 21fa53f406c3de6577719a431818138bc0135fa0..18480befd8938501ab17d21bf792d14670c12c07 100644 (file)
@@ -20,7 +20,7 @@ import com.google.common.base.Preconditions;
 /**
  * Provides augmentation resolving upon given {@link Augmentable}. 
  * Used {@link Augmentation}s do not share {@link Augmentable}.
- * <br/>
+ * <br>
  * <b>Usage:</b> in case there are multiple {@link Augmentable} classes which might contain
  * corresponding {@link Augmentation}s (1:1..n binding). And those {@link Augmentation}s 
  * are sharing the same grouping so that they could be processed in the same way.
@@ -34,7 +34,6 @@ public class GroupingLooseResolver<G> {
 
     /**
      * @param commonInterface
-     * @param common grouping Interface
      */
     public GroupingLooseResolver(Class<G> commonInterface) {
         this.commonInterface = commonInterface;
index 77ac35505636a11d01ae5cb332b04280b6eb44be..c360637d3ce47c31baf57903c1a31496eaad619f 100644 (file)
@@ -19,7 +19,7 @@ import com.google.common.base.Preconditions;
 /**
  * Provides augmentation resolving upon given {@link Augmentable}.
  * Used {@link #classes} share the same {@link Augmentable}.
- * <br/>
+ * <br>
  * <b>Usage:</b> in case there is {@link Augmentable} which might contain
  * multiple {@link Augmentation}s depending on origin. And those {@link Augmentation}s 
  * are sharing the same grouping so that they could be processed in the same way.
@@ -74,4 +74,4 @@ public class GroupingResolver<G, T extends Augmentable<T>> {
         }
         return Optional.absent();
     }
-}
\ No newline at end of file
+}
index 309c5625b132712cda2571bbcae1933d108718bc..466004c2980d192c14991ae27e7ef3cac2783521 100644 (file)
@@ -44,7 +44,7 @@ public interface ExtensionConverterProvider {
     <F extends Action, T extends DataContainer> ConvertorActionToOFJava<F, T> getConverter(TypeVersionKey<F> key);
     
     /**
-     * lookup converter<br/>
+     * lookup converter<br>
      * TODO: this method should be compatible with {@link #getConverter(MessageTypeKey)} after matches are migrated to similar structure
      * @param key
      * @return found converter
index 16e12ff89428e4589b7eb7affaacb00b6ddb105d..e29980894325628d1704ea496f4b17c8138da4e0 100644 (file)
@@ -35,6 +35,8 @@ public enum ActionPath implements AugmentationPath {
 Â Â Â Â  */
     NODES_NODE_TABLE_FLOW_INSTRUCTIONS_INSTRUCTION_WRITEACTIONSCASE_WRITEACTIONS_ACTION_ACTION_EXTENSIONLIST_EXTENSION(null),
     /**
+     * openflowplugin-extension-general.yang
+     * <pre>
      * module: opendaylight-inventory
      *    +--rw nodes
      *       +--rw node* [id]
@@ -69,6 +71,8 @@ public enum ActionPath implements AugmentationPath {
      */
     FLOWSSTATISTICSUPDATE_FLOWANDSTATISTICSMAPLIST_INSTRUCTIONS_INSTRUCTION_INSTRUCTION_WRITEACTIONSCASE_WRITEACTIONS_ACTION_ACTION(null),
     /**
+     * openflowplugin-extension-general.yang
+     * <pre>
      * module: opendaylight-flow-statistics
      * notifications:
      *    +---n flows-statistics-update             
index b7ec91596c8888d72a60e4e41f7b61f383ab1048..4ee26637e1fb3fd3b383ba5a3b07fe16bffaa8f5 100644 (file)
@@ -372,7 +372,7 @@ public class ToSalConversionsUtils {
     }
 
     /**
-     * @param openflow nodeConnector uri
+     * @param uri openflow nodeConnector URI
      * @param node
      * @return assembled nodeConnector
      */
index 3368056332145ead91f940397ea35a495e334b15..62cbf0bdde8c9390a163e00ac7e9711481819862 100644 (file)
@@ -14,7 +14,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflow
 import java.util.Collection;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 27.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 27.3.2015.
  */
 public interface OpenFlowPluginProvider extends AutoCloseable {
 
index 4894f14590c87e7b170a52aa0e8c059c3b31abde..921d1fe53db67a1e3af0d5604ff86f355bd24ed4 100644 (file)
@@ -15,8 +15,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731
 /**
  * Each OpenFlow session is tracked by a Connection Context. These attach to a particular Device Context in such a way,
  * that there is at most one primary session associated with a Device Context.
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface ConnectionContext extends MultiMsgCollector {
 
index 31ce201996cc4e4746982624a55b834f4b560bf9..72589e7e929c6fd0c5297c0fb49e728331e56f5b 100644 (file)
@@ -16,8 +16,8 @@ import org.opendaylight.openflowplugin.api.openflow.device.handlers.MessageHandl
  * Connection manager manages connections with devices.
  * It instantiates and registers {@link ConnectionContext}
  * used for handling all communication with device when onSwitchConnected notification is processed.
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface ConnectionManager extends SwitchConnectionHandler {
 
index 8b7d3e69c6ef9dafc4258e6f5b8a20d9860d032c..b1923e127c3dac5bd37fbe404c301f223d07a6e1 100644 (file)
@@ -33,8 +33,8 @@ import org.opendaylight.yangtools.yang.common.RpcResult;
  * which is enforced by keeping a cap on the number of outstanding requests a particular Request
  * Context can have at any point in time. Should this quota be exceeded, any further attempt to make
  * a request to the switch will fail immediately, with proper error indication.
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface DeviceContext extends MessageHandler {
 
index 196ba7c1d3dcc33fd44b34ed1244b30797c2d1d6..c18375b064d0e6d4c85c4ada742edcc00921912a 100644 (file)
@@ -16,9 +16,8 @@ import org.opendaylight.yangtools.yang.binding.DataObject;
  * This interface is responsible for instantiating DeviceContext and
  * registering transaction chain for each DeviceContext. Each device
  * has its own device context managed by this manager.
- * <p/>
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface DeviceManager extends DeviceConnectedHandler {
 
index d5e926d8d85d210ed8d1bb27b3f572a17bce4c10..fd879ba67d1ecf99861525f31808c0d2e6d53a2f 100644 (file)
@@ -16,7 +16,7 @@ import java.util.Map;
 import java.util.Set;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface DeviceState {
 
index 2122997a729567c3d8784d2e7ad831dae33525f8..269ab3dea408255ef99b3254452a56d1e88f114e 100644 (file)
@@ -13,8 +13,8 @@ import org.opendaylight.yangtools.yang.binding.DataObject;
 /**
  * Request context handles all requests on device. Number of requests is limited by request quota. When this quota is
  * exceeded all rpc's will end up with exception.
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface RequestContext<T extends DataObject> extends RequestFutureContext<T>, AutoCloseable {
 
index dac4ff32a6bd9e5e14895d6f142eaeb78b4a853e..caabf07d67d08dd64feced521c1f7ec4c691a164 100644 (file)
@@ -13,16 +13,15 @@ import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.common.RpcResult;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 25.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.3.2015.
+ * @param <T>
  */
 public interface RequestFutureContext<T extends DataObject> {
 
     /**
      * Method returns future to be used for handling device requests.
      *
-     * @param <T>
      * @return
      */
     SettableFuture<RpcResult<T>> getFuture();
-
 }
index 3a6ee54363f11312d4ad1006c4f032bda6d51488..efd99b536ca1b712d21f08a8e4d63cf979bd2a10 100644 (file)
@@ -9,7 +9,7 @@
 package org.opendaylight.openflowplugin.api.openflow.device;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 26.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 26.2.2015.
  */
 public class Xid {
 
index dfb9802a42402a7cb3dd22f8c0fb81fa564b2591..b17ca8b9839d97a1e7aeb3dfff35b77fab785a47 100644 (file)
@@ -11,7 +11,7 @@ package org.opendaylight.openflowplugin.api.openflow.device;
 import java.util.concurrent.atomic.AtomicLong;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 24.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 24.3.2015.
  */
 public class XidGenerator {
 
index d6c16490644194056acfd383767c13f9dfff2f31..cae8a8515037aef768563c4df5c5c86cd7b3933f 100644 (file)
@@ -9,7 +9,7 @@
 package org.opendaylight.openflowplugin.api.openflow.device.exception;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public class RequestQuotaExceededException extends Exception {
 }
index c10ab6813c590896a294d30d0e4f557f82725f80..24b893f6fb0dd85b82cf5031958ff5ee82516976 100644 (file)
@@ -11,7 +11,7 @@ package org.opendaylight.openflowplugin.api.openflow.device.handlers;
 import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionContext;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 26.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 26.2.2015.
  */
 public interface DeviceConnectedHandler {
 
index 007ab7d9d6da85d14865195eba05741554e48b24..ef39b786d6894c6d333845cbe22b5e83c97f3946 100644 (file)
@@ -11,7 +11,7 @@ package org.opendaylight.openflowplugin.api.openflow.device.handlers;
 import org.opendaylight.openflowplugin.api.openflow.device.DeviceContext;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 27.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 27.2.2015.
  */
 public interface DeviceContextReadyHandler {
     /**
index 970abc31339bf03573d7396f3581165f9d7b39b9..bbcb122edec2b0c24144d2779c6b3107916484d0 100644 (file)
@@ -12,7 +12,7 @@ package org.opendaylight.openflowplugin.api.openflow.device.handlers;
  * This handler does the same, but is not called in the same time as
  * RequestContextReadyHandler.
  *
- * Created by Martin Bobak <mbobak@cisco.com> on 27.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 27.2.2015.
  */
 
 public interface DeviceSynchronizedHandler extends DeviceContextReadyHandler {
index bb4d853bea1b55e0d9dcaf1ed74c734f3244d2cf..c21cb41fff31d49c01b2d02abb3959890551f4e3 100644 (file)
@@ -13,7 +13,7 @@ import org.opendaylight.yangtools.yang.binding.ChildOf;
 import org.opendaylight.yangtools.yang.binding.DataObject;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 26.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 26.2.2015.
  */
 public interface MessageHandler {
 
index 4a52c3b96d44202a7d75cc3c563d559735ec0106..a4a0abeda5f5a67d1cf16da4e5a60fb2bd1f120e 100644 (file)
@@ -41,7 +41,7 @@ public interface ConnectionConductor {
     List<Short> versionOrder = Lists.newArrayList((short) 0x04, (short) 0x01);
 
     /**
-     * initialize wiring around {@link #connectionAdapter}
+     * initialize wiring around {@link ConnectionAdapter}
      */
     void init();
 
index 7eece60f7186bd549b8d19e90a38e33cb2896a49..8176a5fab388a667b1eb541d865ddbae273221eb 100644 (file)
@@ -13,14 +13,13 @@ import org.opendaylight.openflowplugin.api.openflow.md.core.ConnectionConductor;
  * This processing mechanism based on queue. Processing consists of 2 steps: translate and publish.
  * Proposed workflow (might slightly deviate in implementations):
  * <ol>
- * <li>messages of input type are pushed in (via {@link QueueKeeper#push(Object, ConnectionConductor)} and similar)</li>
+ * <li>messages of input type are pushed in (via {@link QueueKeeper#push(Object, ConnectionConductor, QueueType)} and similar)</li>
  * <li>ticket (executable task) is build upon each pushed message and enqueued</li>
  * <li>ticket is translated using appropriate translator</li>
  * <li>ticket is dequeued and result is published by appropriate popListener</li>
  * </ol>
  * Message order might be not important, e.g. when speed is of the essence
  * @param <I> source type (IN)
- * @param <O> result type (OUT)
  */
 public interface QueueKeeper<I> extends AutoCloseable {
 
index 226ccba422089007b328a58dc7ef3b5c540391a9..441bb9b0a22a0164bd5be130ee6f944abcac4026 100644 (file)
@@ -11,7 +11,6 @@ import java.util.Collection;
 import java.util.List;
 import java.util.Map;
 
-import org.opendaylight.openflowplugin.api.openflow.md.core.ConnectionConductor;
 import org.opendaylight.openflowplugin.api.openflow.md.core.IMDMessageTranslator;
 import org.opendaylight.openflowplugin.api.openflow.md.core.TranslatorKey;
 
@@ -19,7 +18,7 @@ import org.opendaylight.openflowplugin.api.openflow.md.core.TranslatorKey;
  * This processing mechanism based on queue. Processing consists of 2 steps: translate and publish.
  * Proposed workflow (might slightly deviate in implementations):
  * <ol>
- * <li>messages of input type are pushed in (via {@link QueueProcessor#push(Object, ConnectionConductor)} and similar)</li>
+ * <li>messages of input type are pushed in (via {@link QueueKeeper#push(Object, org.opendaylight.openflowplugin.api.openflow.md.core.ConnectionConductor, org.opendaylight.openflowplugin.api.openflow.md.queue.QueueKeeper.QueueType)} and similar)</li>
  * <li>ticket (executable task) is build upon each pushed message and enqueued</li>
  * <li>ticket is translated using appropriate translator</li>
  * <li>ticket is dequeued and result is published by appropriate popListener</li>
index 2547f8315dd964deafca2ec1ab950b2c260073c7..7baa5a9d4d6535dfee32d901dcb56167fd2e749b 100644 (file)
@@ -1,7 +1,7 @@
 package org.opendaylight.openflowplugin.api.openflow.md.util;
 
 /** List of Openflow versions supported by the plugin
- *  Note: If you add a version here, make sure to update {@link OpenflowPortsUtil} as well.
+ *  Note: If you add a version here, make sure to update org.opendaylight.openflowplugin.openflow.md.util.OpenflowPortsUtil as well.
  * Created by kramesha on 5/2/14.
  */
 public enum OpenflowVersion {
index f28876eded2a053ea11a99492b54051ab59f1ba3..d853236cee42f657854b0d814acf582a3f793220 100644 (file)
@@ -13,12 +13,11 @@ import java.util.Queue;
  * Zipper groups together a list of queues and exposes one poll method. Polling
  * iterates through all groups and returns first not-null result of poll method
  * on each queue. If after polling each grouped queue for one time there is
- * still null result, poll will return null. <br/>
+ * still null result, poll will return null. <br>
  * Iterating keeps last position so this polling is supposed to be fairly
  * distributed.
  * 
- * @param <T>
- *            common item type of zipped queues
+ * @param <T> common item type of zipped queues
  */
 public class PollableQueuesPriorityZipper<T> {
 
index c869c64ab072b43d5626aba183deeef876a02bbf..4de64b31f3d1383c04f3e0a4f45d69618225376d 100644 (file)
@@ -18,7 +18,7 @@ import com.google.common.collect.Iterators;
  * Zipper groups together a list of queues and exposes one poll method. Polling iterates through
  * all groups and returns first not-null result of poll method on each queue. If after polling each 
  * grouped queue for one time there is still null result, poll will return null. 
- * <br/>
+ * <br>
  * Iterating keeps last position so this polling is supposed to be fairly distributed.
  * 
  * @param <T> common item type of zipped queues
index 651fe6931d469a5811cb913976a790f5740dd19e..8a3fc48c74d934deca1fa16d44626d035b57e433 100644 (file)
@@ -18,8 +18,8 @@ import org.opendaylight.yangtools.yang.common.RpcResult;
  * This context is registered with MD-SAL as a routed RPC provider for the inventory node backed by this switch and
  * tracks the state of any user requests and how they map onto protocol requests. It uses
  * {@link org.opendaylight.openflowplugin.api.openflow.device.RequestContext} to perform requests.
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface RpcContext extends AutoCloseable {
 
index 249184aac3dd82b6835b9b992b4dfaa2f58adfcf..72fae2bb0ba622b75051045a2a9d2bf7ea0226e1 100644 (file)
@@ -12,10 +12,10 @@ import org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceSynchr
 
 /**
  * The RPC Manager will maintain an RPC Context for each online switch. RPC context for device is created when
- * {@link org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceSynchronizedHandler#deviceConnected(org.opendaylight.openflowplugin.api.openflow.device.RequestContext)}
+ * {@link org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceSynchronizedHandler#deviceConnected(org.opendaylight.openflowplugin.api.openflow.device.DeviceContext)}
  * is called.
- * <p/>
- * Created by Martin Bobak <mbobak@cisco.com> on 25.2.2015.
+ * <p>
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 25.2.2015.
  */
 public interface RpcManager extends DeviceSynchronizedHandler {
 
index eeb55c65be50cab5e1bf5096ed169991681ad75d..919e8d55ef3f4c6c624ba712d0a91327c3d5457d 100644 (file)
@@ -12,7 +12,7 @@ import org.opendaylight.openflowplugin.api.openflow.device.DeviceContext;
 import org.opendaylight.openflowplugin.api.openflow.device.RequestContext;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 27.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 27.2.2015.
  */
 public interface StatisticsContext {
 
index 99dbcc93853ff5971ab774be9581d902f3ced2ad..bf75a10fff09aa6cedd995d2e80601f7f70a78a0 100644 (file)
@@ -11,7 +11,7 @@ package org.opendaylight.openflowplugin.api.openflow.statistics;
 import org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceContextReadyHandler;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 26.2.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 26.2.2015.
  */
 public interface StatisticsManager extends DeviceContextReadyHandler {
 
index 3a17dee45386101c25a70009b57b6d0b4684452e..7b269822893e41c2975c7c7ad1f16372339ceda8 100644 (file)
@@ -21,7 +21,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflow
 import java.util.Collection;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 27.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 27.3.2015.
  */
 public class OpenflowPluginProviderImpl implements OpenFlowPluginProvider {
 
index da097a5cd9080cd4e0d7d0a8719fef8a51e6e680..b8aa9949fe80d1f17853769d579dcd109ed744c8 100644 (file)
@@ -13,7 +13,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.common.types.rev13
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.MultipartRequestInputBuilder;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 26.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 26.3.2015.
  */
 public final class RequestInputUtils {
 
index 0429c055f4cf73add146d6e3906fc83d004f1375..abd4b2aee0cdf77b54307b56fa767135bd74bf32 100644 (file)
@@ -20,7 +20,7 @@ import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 26.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 26.3.2015.
  */
 public class RpcResultConvertor<T extends DataObject> {
 
index a9f5a0d62edbc6a52ae3fdb7bb2d67da49fe0f4e..b108d336380b8429d28a512c53d9a7b19873220e 100644 (file)
@@ -305,7 +305,7 @@ public class HandshakeManagerImpl implements HandshakeManager {
     /**
      * after handshake set features, register to session
      * @param proposedVersion
-     * @param xId
+     * @param xid
      * @throws Exception 
      */
     protected void postHandshake(Short proposedVersion, Long xid) throws Exception {
index 371ed9e04820c2ac0b64a984dbd669c45119ddb5..226562f2fdf614f256818be8b11b68938cbac9c6 100644 (file)
@@ -228,8 +228,8 @@ public abstract class OFRpcTaskFactory {
     }
 
     /**
-     * Recursive helper method for {@link OFRpcTaskFactory#createAddFlowTask()}
-     * and {@link OFRpcTaskFactory#createUpdateFlowTask()} to chain results
+     * Recursive helper method for {@link OFRpcTaskFactory#createAddFlowTask(OFRpcTaskContext, AddFlowInput, SwitchConnectionDistinguisher)}
+     * and {@link OFRpcTaskFactory#createUpdateFlowTask(OFRpcTaskContext, UpdateFlowInput, SwitchConnectionDistinguisher)} to chain results
      * of multiple flowmods.
      * The next flowmod gets executed if the earlier one is successful.
      * All the flowmods should have the same xid, in-order to cross-reference
@@ -357,7 +357,6 @@ public abstract class OFRpcTaskFactory {
 
 
     /**
-     * @param xId
      * @param input
      * @return
      */
index aa59636598beb8613267759fa05b0c1b88afc6bd..1cd5d8a51a9daa464cf679d5ce6705794334a59e 100644 (file)
@@ -135,8 +135,6 @@ public abstract class OFRpcTaskUtil {
     /**
      * @param task of rpcl
      * @param originalResult
-     * @param notificationProviderService
-     * @param notificationComposer lazy notification composer
      * @return chained result with barrier
      */
     public static <T extends TransactionAware, I extends DataContainer>
index 49af9ea9f1059bbe07d291308a42d563cd30b40c..60fcc5da515b167a9a956e0e59ca76280c694258 100644 (file)
@@ -9,7 +9,7 @@
 package org.opendaylight.openflowplugin.openflow.md.core.sal.convertor;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 13.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 13.3.2015.
  */
 public final class ActionUtil {
 
index ea1948c2d3c511eb978e0282c5789b564a930eec..d4b5f55358f53268f6c85b8135d9b7609ca61b23 100644 (file)
@@ -17,7 +17,7 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.
 import java.util.Iterator;
 
 /**
- * Created by Martin Bobak <mbobak@cisco.com> on 5.3.2015.
+ * Created by Martin Bobak &lt;mbobak@cisco.com&gt; on 5.3.2015.
  */
 public final class IpConversionUtil {
 
index 6a25786d749698a83bf25449785a7c551de477ab..403422e4810146a9b66bd48e4a3b26bfd5d6a54d 100644 (file)
@@ -32,7 +32,6 @@ public class QueueKeeperHarvester<IN> implements Runnable, HarvesterHandle {
     /**
      * @param enqueuer
      * @param messageSources
-     * @param harvestLock
      */
     public QueueKeeperHarvester(final Enqueuer<QueueItem<IN>> enqueuer,
             final Collection<QueueKeeper<IN>> messageSources) {
@@ -89,4 +88,4 @@ public class QueueKeeperHarvester<IN> implements Runnable, HarvesterHandle {
             }
         }
     }
-}
\ No newline at end of file
+}
index 2df8fb229efcb7e6b24672ae86ad39bf48338f15..63fa440d6c70a5b30280236a495f8c98aecb2b23 100644 (file)
@@ -38,15 +38,15 @@ import org.slf4j.LoggerFactory;
 
 /**
  * {@link org.opendaylight.openflowplugin.api.openflow.md.queue.QueueKeeper} implementation focused to keep order and use up mutiple threads for translation phase.
- * <br/>
+ * <br>
  * There is internal thread pool of limited size ({@link QueueProcessorLightImpl#setProcessingPoolSize(int)})
  * dedicated to translation. Then there is singleThreadPool dedicated to publishing (via popListeners)
- * <br/>
+ * <br>
  * Workflow:
  * <ol>
  * <li>upon message push ticket is created and enqueued</li>
  * <li>available threads from internal pool translate the massage wrapped in ticket</li>
- * <li>when translation of particular message is finished, result is set in future result of wrapping ticket</br>
+ * <li>when translation of particular message is finished, result is set in future result of wrapping ticket<br>
  *     (order of tickets in queue is not touched during translate)
  * </li>
  * <li>at the end of queue there is {@link TicketFinisher} running in singleThreadPool and for each ticket it does:
index 8a3404f923f16273173e83155826c9d21b343e72..10b3b2c5339b9627c737253c4d8403bb453098f1 100644 (file)
@@ -27,7 +27,6 @@ public class WrapperQueueImpl<E> implements Queue<E> {
 
     /**
      * @param capacity
-     * @param queueZipper
      */
     public WrapperQueueImpl(int capacity, Queue<E> queueDefault,
             WaterMarkListener queueListenerMark) {
index 9ee9a1e681eb5f024da41c996a81d4a99c8a84db..cd308d59b2f4f903c2a8a7a0180c6afb98a84a6a 100644 (file)
@@ -19,7 +19,7 @@ import org.slf4j.LoggerFactory;
 
 /**
  * learning switch activator
- * <p/>
+ * <p>
  * Activator is derived from AbstractBindingAwareConsumer, which takes care
  * of looking up MD-SAL in Service Registry and registering consumer
  * when MD-SAL is present.
index 18f71bb0fb589317909d850062b6746b37f2e4f5..10ee47ab6369dd0ed01eb6c0cd2e539eae8fba8f 100644 (file)
@@ -61,12 +61,12 @@ public class InstanceIdentifierUtils {
      * Creates a path for particular flow, by appending flow-specific information
      * to table path.
      * 
-     * @param flowId
-     * @param tablePathArg 
+     * @param tablePath
+     * @param flowKey
      * @return path to flow
      */
-    public static InstanceIdentifier<Flow> createFlowPath(InstanceIdentifier<Table> table,FlowKey flowKey) {
-        return InstanceIdentifier.builder(table)
+    public static InstanceIdentifier<Flow> createFlowPath(InstanceIdentifier<Table> tablePath, FlowKey flowKey) {
+        return InstanceIdentifier.builder(tablePath)
                 .child(Flow.class, flowKey)
                 .build();
     }
index 0ba0ed22e9ad7f37c497efda421704e2bf6162d7..56c577899f2aa818c46fd8cfdb5f4b374dc64254 100644 (file)
@@ -29,14 +29,14 @@ public interface LearningSwitchManager {
 
     /**
      * Set's Data Broker dependency.
-     * <p/>
+     * <p>
      * Data Broker is used to access overal operational and configuration
      * tree.
-     * <p/>
+     * <p>
      * In simple Learning Switch handler, data broker is used to listen
      * for changes in Openflow tables and to configure flows which will
      * be provisioned down to the Openflow switch.
-     * <p/>
+     * <p>
      * inject {@link DataBroker}
      *
      * @param data
@@ -45,10 +45,10 @@ public interface LearningSwitchManager {
 
     /**
      * Set's Packet Processing dependency.
-     * <p/>
+     * <p>
      * Packet Processing service is used to send packet Out on Openflow
      * switch.
-     * <p/>
+     * <p>
      * inject {@link PacketProcessingService}
      *
      * @param packetProcessingService
@@ -58,10 +58,10 @@ public interface LearningSwitchManager {
 
     /**
      * Set's Notification service dependency.
-     * <p/>
+     * <p>
      * Notification service is used to register for listening
      * packet-in notifications.
-     * <p/>
+     * <p>
      * inject {@link NotificationService}
      *
      * @param notificationService
index 5bbefe204824e4cdf80e8524ccd4752d93287183..908a2fe5937a97645f732b231644379e53f38af9 100644 (file)
@@ -3257,7 +3257,7 @@ public class OpenflowpluginTestCommandProvider implements CommandProvider {
 
     /**
      * @param ci arguments: switchId flowType tableNum
-     *           <p/>
+     *           <p>
      *           <pre>
      *                     e.g.: addMDFlow openflow:1 f1 42
      *                     </pre>