Fix checkstyle violations in sal-dom-api 03/69103/2
authorTom Pantelis <tompantelis@gmail.com>
Tue, 6 Mar 2018 01:59:41 +0000 (20:59 -0500)
committerMichael Vorburger <vorburger@redhat.com>
Wed, 7 Mar 2018 12:49:40 +0000 (12:49 +0000)
Change-Id: I2be21dbde5cca70ca327828fd479779ceb958c3e
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
35 files changed:
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/ClusteredDOMDataChangeListener.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/ClusteredDOMDataTreeChangeListener.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataBroker.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataReadOnlyTransaction.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataReadTransaction.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataReadWriteTransaction.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeChangeListener.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeChangeService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeIdentifier.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeInaccessibleException.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeListener.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeProducer.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataTreeShardingService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMDataWriteTransaction.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMEvent.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMExtensibleService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMMountPoint.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMMountPointService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMNotificationPublishService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMNotificationRejectedException.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMNotificationService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMRpcIdentifier.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMRpcImplementationNotAvailableException.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMTransactionChain.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/AbstractConsumer.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/AbstractProvider.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/Broker.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/BrokerService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/Consumer.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/Provider.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/RpcRoutingContext.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/model/SchemaService.java
opendaylight/md-sal/sal-dom-api/src/test/java/org/opendaylight/controller/md/sal/dom/api/AbstractDOMDataTreeServiceTestSuite.java
opendaylight/md-sal/sal-test-model/src/main/java/org/opendaylight/controller/md/sal/test/model/util/ListsBindingUtils.java

index e5dbc545e524f8679cd25a4feb4cadcb59cd38f3..334ca95d91ca89156307524da99f70f1c047ee3a 100644 (file)
@@ -9,16 +9,14 @@
 package org.opendaylight.controller.md.sal.dom.api;
 
 /**
- * <p>ClusteredDOMDataChangeListener is a marker interface to enable data change notifications on all instances in a cluster,
- * where this listener is registered.</p>
- *
- *
- * <p>Applications should implement ClusteredDOMDataChangeListener instead of DOMDataChangeListener, if they want to listen
- * to data change notifications on any node of clustered datastore. DOMDataChangeListener enables data change notifications
- * only at leader of the datastore shard.</p>
+ * ClusteredDOMDataChangeListener is a marker interface to enable data change
+ * notifications on all instances in a cluster, where this listener is
+ * registered.
  *
+ * <p>
+ * Applications should implement ClusteredDOMDataChangeListener instead of DOMDataChangeListener, if they want to
+ * listen to data change notifications on any node of clustered datastore. DOMDataChangeListener enables data change
+ * notifications only at leader of the datastore shard.
  */
-
 public interface ClusteredDOMDataChangeListener extends DOMDataChangeListener{
-
 }
index 5da80e4907f5bcf59b25be11c2205aa9c155c2f2..3fdc1b4eee4b86d5f49ac1ada648caf0dc06a04c 100644 (file)
@@ -8,8 +8,9 @@
 package org.opendaylight.controller.md.sal.dom.api;
 
 /**
- *  ClusteredDOMDataTreeChangeListener is a marker interface to enable data tree change notifications on all
+ * ClusteredDOMDataTreeChangeListener is a marker interface to enable data tree change notifications on all
  * instances in a cluster where this listener is registered.
+ *
  * <p>
  * Applications should implement ClusteredDOMDataTreeChangeListener instead of {@link DOMDataTreeChangeListener},
  * if they want to listen for data tree change notifications on any node of a clustered data store.
index 0aba3ca8b4617e8f53c5d1bcdb2cfb0fd4e9818a..bf2758fec3fc29dc7c2a84d163e2bb6944d38e6b 100644 (file)
@@ -19,35 +19,25 @@ import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
  * Data Broker which provides data transaction and data change listener functionality
  * using {@link NormalizedNode} data format.
  *
+ * <p>
  * This interface is type capture of generic interfaces and returns type captures
  * of results for client-code convenience.
  *
  */
 public interface DOMDataBroker extends
         AsyncDataBroker<YangInstanceIdentifier, NormalizedNode<?, ?>, DOMDataChangeListener>,
-        TransactionChainFactory<YangInstanceIdentifier, NormalizedNode<?, ?>>, BrokerService, DOMExtensibleService<DOMDataBroker, DOMDataBrokerExtension> {
+        TransactionChainFactory<YangInstanceIdentifier, NormalizedNode<?, ?>>, BrokerService,
+            DOMExtensibleService<DOMDataBroker, DOMDataBrokerExtension> {
 
-    /**
-     * {@inheritDoc}
-     */
     @Override
     DOMDataReadOnlyTransaction newReadOnlyTransaction();
 
-    /**
-     * {@inheritDoc}
-     */
     @Override
     DOMDataReadWriteTransaction newReadWriteTransaction();
 
-    /**
-     * {@inheritDoc}
-     */
     @Override
     DOMDataWriteTransaction newWriteOnlyTransaction();
 
-    /**
-     * {@inheritDoc}
-     */
     @Override
     DOMTransactionChain createTransactionChain(TransactionChainListener listener);
 }
index 41fb7abc5e1081f92128f987a37792a8f563a6c3..53af4551f91a081f0d62cb47e3da884860d0255f 100644 (file)
@@ -12,6 +12,6 @@ import org.opendaylight.controller.md.sal.common.api.data.AsyncReadOnlyTransacti
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 
-public interface DOMDataReadOnlyTransaction extends DOMDataReadTransaction, AsyncReadOnlyTransaction<YangInstanceIdentifier, NormalizedNode<?, ?>> {
-
+public interface DOMDataReadOnlyTransaction extends DOMDataReadTransaction,
+        AsyncReadOnlyTransaction<YangInstanceIdentifier, NormalizedNode<?, ?>> {
 }
index 05ca61a3535699273ed4ca0e169a9fcbd9dc88b7..111a503890f9e14fe8d3c8a1d70c3d739f5621d0 100644 (file)
@@ -7,17 +7,17 @@
  */
 package org.opendaylight.controller.md.sal.dom.api;
 
+import com.google.common.base.Optional;
+import com.google.common.util.concurrent.CheckedFuture;
 import org.opendaylight.controller.md.sal.common.api.data.AsyncReadTransaction;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.controller.md.sal.common.api.data.ReadFailedException;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 
-import com.google.common.base.Optional;
-import com.google.common.util.concurrent.CheckedFuture;
-
 /**
  * A transaction that provides read access to a logical data store.
+ *
  * <p>
  * For more information on usage and examples, please see the documentation in {@link AsyncReadTransaction}.
  */
@@ -25,7 +25,8 @@ public interface DOMDataReadTransaction extends AsyncReadTransaction<YangInstanc
 
     /**
      * Reads data from provided logical data store located at the provided path.
-     *<p>
+     *
+     * <p>
      * If the target is a subtree, then the whole subtree is read (and will be
      * accessible from the returned data object).
      *
@@ -51,8 +52,8 @@ public interface DOMDataReadTransaction extends AsyncReadTransaction<YangInstanc
     /**
      /**
      * Checks if data is available in the logical data store located at provided path.
-     * <p>
      *
+     * <p>
      * Note: a successful result from this method makes no guarantee that a subsequent call to {@link #read}
      * will succeed. It is possible that the data resides in a data store on a remote node and, if that
      * node goes down or a network failure occurs, a subsequent read would fail. Another scenario is if
@@ -71,7 +72,5 @@ public interface DOMDataReadTransaction extends AsyncReadTransaction<YangInstanc
      *         {@link ReadFailedException} or an exception derived from ReadFailedException.</li>
      *         </ul>
      */
-    CheckedFuture<Boolean, ReadFailedException> exists(
-        LogicalDatastoreType store, YangInstanceIdentifier path);
-
+    CheckedFuture<Boolean, ReadFailedException> exists(LogicalDatastoreType store, YangInstanceIdentifier path);
 }
index d5fc5c957c852e563a82fb7871fcbee3f858d0f8..39e2ad3ac6ef4e74076193af72524cd0593980d1 100644 (file)
@@ -11,6 +11,6 @@ import org.opendaylight.controller.md.sal.common.api.data.AsyncReadWriteTransact
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 
-public interface DOMDataReadWriteTransaction extends DOMDataReadTransaction, DOMDataWriteTransaction, AsyncReadWriteTransaction<YangInstanceIdentifier, NormalizedNode<?, ?>> {
-
+public interface DOMDataReadWriteTransaction extends DOMDataReadTransaction, DOMDataWriteTransaction,
+        AsyncReadWriteTransaction<YangInstanceIdentifier, NormalizedNode<?, ?>> {
 }
index b922a5db29b4cd89d7d282137677438db7106701..dfefc273a74a6f7ffa6c99e538d2aca953a90d59 100644 (file)
@@ -17,6 +17,7 @@ import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidate;
  * data tree changes. This interface differs from {@link DOMDataChangeListener}
  * in that it provides a cursor-based view of the change, which has potentially
  * lower overhead.
+ *
  * <p>
  * Note: this interface enables notifications only at the leader of the data store, if clustered. If you want
  * notifications on all instances in a cluster, use the {@link ClusteredDOMDataTreeChangeListener}.
@@ -37,6 +38,7 @@ public interface DOMDataTreeChangeListener extends EventListener {
      * to recover from such events. Event producers are expected to exert reasonable
      * effort to suppress such events.
      *
+     * <p>
      * In other words, it is completely acceptable to observe
      * a {@link org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidateNode},
      * which reports a {@link org.opendaylight.yangtools.yang.data.api.schema.tree.ModificationType}
index bdd853a1c08a033a6f2768f20b56eca0c2bf9977..46a6fb2cb7ccf6bf140689ce6ed10ecf715df1bd 100644 (file)
@@ -19,14 +19,16 @@ public interface DOMDataTreeChangeService extends DOMDataBrokerExtension {
      * Registers a {@link DOMDataTreeChangeListener} to receive
      * notifications when data changes under a given path in the conceptual data
      * tree.
+     *
      * <p>
      * You are able to register for notifications  for any node or subtree
      * which can be represented using {@link DOMDataTreeIdentifier}.
-     * <p>
      *
+     * <p>
      * You are able to register for data change notifications for a subtree or leaf
      * even if it does not exist. You will receive notification once that node is
      * created.
+     *
      * <p>
      * If there is any pre-existing data in the data tree for the path for which you are
      * registering, you will receive an initial data change event, which will
@@ -36,6 +38,7 @@ public interface DOMDataTreeChangeService extends DOMDataBrokerExtension {
      * This method returns a {@link ListenerRegistration} object. To
      * "unregister" your listener for changes call the {@link ListenerRegistration#close()}
      * method on the returned object.
+     *
      * <p>
      * You MUST explicitly unregister your listener when you no longer want to receive
      * notifications. This is especially true in OSGi environments, where failure to
@@ -50,5 +53,6 @@ public interface DOMDataTreeChangeService extends DOMDataBrokerExtension {
      *         your listener using {@link ListenerRegistration#close()} to stop
      *         delivery of change events.
      */
-    @Nonnull <L extends DOMDataTreeChangeListener> ListenerRegistration<L> registerDataTreeChangeListener(@Nonnull DOMDataTreeIdentifier treeId, @Nonnull L listener);
+    @Nonnull <L extends DOMDataTreeChangeListener> ListenerRegistration<L> registerDataTreeChangeListener(
+            @Nonnull DOMDataTreeIdentifier treeId, @Nonnull L listener);
 }
index 407c466c3998a168c6a83cf55556e62c5245c8d8..0f015d1348014d2a627724ee965a28f84a7d35b0 100644 (file)
@@ -23,12 +23,14 @@ import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.PathArgum
  * A unique identifier for a particular subtree. It is composed of the logical
  * data store type and the instance identifier of the root node.
  */
-public final class DOMDataTreeIdentifier implements Immutable, Path<DOMDataTreeIdentifier>, Serializable, Comparable<DOMDataTreeIdentifier> {
+public final class DOMDataTreeIdentifier implements Immutable,
+        Path<DOMDataTreeIdentifier>, Serializable, Comparable<DOMDataTreeIdentifier> {
     private static final long serialVersionUID = 1L;
     private final YangInstanceIdentifier rootIdentifier;
     private final LogicalDatastoreType datastoreType;
 
-    public DOMDataTreeIdentifier(final LogicalDatastoreType datastoreType, final YangInstanceIdentifier rootIdentifier) {
+    public DOMDataTreeIdentifier(final LogicalDatastoreType datastoreType,
+            final YangInstanceIdentifier rootIdentifier) {
         this.datastoreType = Preconditions.checkNotNull(datastoreType);
         this.rootIdentifier = Preconditions.checkNotNull(rootIdentifier);
     }
@@ -81,14 +83,14 @@ public final class DOMDataTreeIdentifier implements Immutable, Path<DOMDataTreeI
     }
 
     @Override
-    public int compareTo(final DOMDataTreeIdentifier o) {
-        int i = datastoreType.compareTo(o.datastoreType);
-        if (i != 0) {
-            return i;
+    public int compareTo(final DOMDataTreeIdentifier obj) {
+        int cmp = datastoreType.compareTo(obj.datastoreType);
+        if (cmp != 0) {
+            return cmp;
         }
 
         final Iterator<PathArgument> mi = rootIdentifier.getPathArguments().iterator();
-        final Iterator<PathArgument> oi = o.rootIdentifier.getPathArguments().iterator();
+        final Iterator<PathArgument> oi = obj.rootIdentifier.getPathArguments().iterator();
 
         while (mi.hasNext()) {
             if (!oi.hasNext()) {
@@ -97,9 +99,9 @@ public final class DOMDataTreeIdentifier implements Immutable, Path<DOMDataTreeI
 
             final PathArgument ma = mi.next();
             final PathArgument oa = oi.next();
-            i = ma.compareTo(oa);
-            if (i != 0) {
-                return i;
+            cmp = ma.compareTo(oa);
+            if (cmp != 0) {
+                return cmp;
             }
         }
 
index 2f8eeb0c2ad2ab40b597bc15389d124a26c1f6aa..8a6b8dacc7be4159a9a4cd23061bb42d9148af9b 100644 (file)
@@ -24,7 +24,8 @@ public class DOMDataTreeInaccessibleException extends DOMDataTreeListeningExcept
         this.treeIdentifier = Preconditions.checkNotNull(treeIdentifier);
     }
 
-    public DOMDataTreeInaccessibleException(final DOMDataTreeIdentifier treeIdentifier, final String message, final Throwable cause) {
+    public DOMDataTreeInaccessibleException(final DOMDataTreeIdentifier treeIdentifier, final String message,
+            final Throwable cause) {
         super(message);
         this.treeIdentifier = Preconditions.checkNotNull(treeIdentifier);
     }
index 5431c861387efa825f35c19bf664fa22a3b55c1a..5a3de720c5a8342507dcc88359827efb98523c6a 100644 (file)
@@ -32,7 +32,8 @@ public interface DOMDataTreeListener extends EventListener {
      *                 This includes all the subtrees this listener is subscribed to, even those
      *                 which have not changed.
      */
-    void onDataTreeChanged(@Nonnull Collection<DataTreeCandidate> changes, @Nonnull Map<DOMDataTreeIdentifier, NormalizedNode<?, ?>> subtrees);
+    void onDataTreeChanged(@Nonnull Collection<DataTreeCandidate> changes,
+            @Nonnull Map<DOMDataTreeIdentifier, NormalizedNode<?, ?>> subtrees);
 
     /**
      * Invoked when a subtree listening failure occurs. This can be triggered, for example, when
index 404de1c00f794747f3ab17c2a63dba69e40f582e..1a91a0af9f203b7e7af801684c7d04e94b01dd28 100644 (file)
@@ -16,21 +16,26 @@ import javax.annotation.Nonnull;
  * open. It needs to be either submitted or cancelled before another one can be open.
  * Once a transaction is submitted, it will proceed to be committed asynchronously.
  *
+ * <p>
  * Each instance has  an upper bound on the number of transactions which can be in-flight,
  * once that capacity is exceeded, an attempt to create a new transaction will block
  * until some transactions complete.
  *
+ * <p>
  * Each {@link DOMDataTreeProducer} can be in two logical states, bound and unbound,
  * which define the lifecycle rules for when is it legal to create and submit transactions
  * in relationship with {@link DOMDataTreeListener} callbacks.
  *
+ * <p>
  * When a producer is first created, it is unbound. In this state the producer can be
  * accessed by any application thread to allocate or submit transactions, as long as
  * the 'single open transaction' rule is maintained. The producer and any transaction
  * object MUST NOT be accessed, directly or indirectly, from a {@link DOMDataTreeListener}
  * callback.
  *
- * When a producer is referenced in a call to {@link DOMDataTreeService#registerListener(DOMDataTreeListener, java.util.Collection, boolean, java.util.Collection)},
+ * <p>
+ * When a producer is referenced in a call to {@link DOMDataTreeService#registerListener(DOMDataTreeListener,
+ * java.util.Collection, boolean, java.util.Collection)},
  * an attempt will be made to bind the producer to the specified {@link DOMDataTreeListener}.
  * Such an attempt will fail the producer is already bound, or it has an open transaction.
  * Once bound, the producer can only be accessed from within the {@link DOMDataTreeListener}
@@ -64,15 +69,18 @@ public interface DOMDataTreeProducer extends DOMDataTreeProducerFactory, AutoClo
     /**
      * {@inheritDoc}
      *
+     * <p>
      * When invoked on a {@link DOMDataTreeProducer}, this method has additional restrictions.
      * There may not be an open transaction from this producer. The method needs to be
      * invoked in appropriate context, e.g. bound or unbound.
      *
+     * <p>
      * Specified subtrees must be accessible by this producer. Accessible means they are a subset
      * of the subtrees specified when the producer is instantiated. The set is further reduced as
      * child producers are instantiated -- if you create a producer for /a and then a child for
      * /a/b, /a/b is not accessible from the first producer.
      *
+     * <p>
      * Once this method returns successfully, this (parent) producer loses the ability to
      * access the specified paths until the resulting (child) producer is shut down.
      *
index 5323cea7470e53bc2a6cd564838cc6d1f9df4289..e39d49f11763b537bafea69c4050587d7a983852 100644 (file)
@@ -16,10 +16,12 @@ import org.opendaylight.yangtools.concepts.ListenerRegistration;
  * with the data tree are split into data producers and consumers (listeners). Each
  * of them operate on a set of subtrees, which need to be declared at instantiation time.
  *
+ * <p>
  * Returned instances are not thread-safe and expected to be used by a single thread
  * at a time. Furthermore, producers may not be accessed from consumer callbacks
  * unless they were specified when the listener is registered.
  *
+ * <p>
  * The service maintains a loop-free topology of producers and consumers. What this means
  * is that a consumer is not allowed to access a producer, which affects any of the
  * subtrees it is subscribed to. This restriction is in place to ensure the system does
@@ -38,12 +40,14 @@ public interface DOMDataTreeService extends DOMDataTreeProducerFactory, DOMServi
      * is free to merge the changes, so that a smaller number of them will be reported,
      * possibly hiding some data transitions (like flaps).
      *
+     * <p>
      * If the listener wants to write into any producer, that producer has to be mentioned
      * in the call to this method. Those producers will be bound exclusively to the
      * registration, so that accessing them outside of this listener's callback will trigger
      * an error. Any producers mentioned must be idle, e.g. they may not have an open
      * transaction at the time this method is invoked.
      *
+     * <p>
      * Each listener instance can be registered at most once. Implementations of this
      * interface have to guarantee that the listener's methods will not be invoked
      * concurrently from multiple threads.
@@ -61,5 +65,6 @@ public interface DOMDataTreeService extends DOMDataTreeProducerFactory, DOMServi
      *                                  feedback loop
      */
     @Nonnull <T extends DOMDataTreeListener> ListenerRegistration<T> registerListener(@Nonnull T listener,
-        @Nonnull Collection<DOMDataTreeIdentifier> subtrees, boolean allowRxMerges, @Nonnull Collection<DOMDataTreeProducer> producers) throws DOMDataTreeLoopException;
+        @Nonnull Collection<DOMDataTreeIdentifier> subtrees, boolean allowRxMerges,
+        @Nonnull Collection<DOMDataTreeProducer> producers) throws DOMDataTreeLoopException;
 }
index e8c09a32fb8e9e06ee5175e06e2a85b328ac7559..fea4cdd61afb772afa7af450e6af0fec2d6873a6 100644 (file)
@@ -16,11 +16,13 @@ import org.opendaylight.yangtools.concepts.ListenerRegistration;
  * single {@link DOMDataTreeIdentifier}. Based on those data tree identifiers, the
  * shards are organized in a tree, where there is a logical parent/child relationship.
  *
+ * <p>
  * It is not allowed to attach two shards to the same data tree identifier, which means
  * the mapping of each piece of information has an unambiguous home. When accessing
  * the information, the shard with the longest matching data tree identifier is used,
  * which is why this interface treats it is a prefix.
  *
+ * <p>
  * Whenever a parent/child relationship is changed, the parent is notified, so it can
  * understand that a logical child has been attached.
  *
@@ -36,5 +38,6 @@ public interface DOMDataTreeShardingService extends DOMService {
      * @return A registration. To remove the shard's binding, close the registration.
      * @throws DOMDataTreeShardingConflictException if the prefix is already bound
      */
-    @Nonnull <T extends DOMDataTreeShard> ListenerRegistration<T> registerDataTreeShard(@Nonnull DOMDataTreeIdentifier prefix, @Nonnull T shard) throws DOMDataTreeShardingConflictException;
+    @Nonnull <T extends DOMDataTreeShard> ListenerRegistration<T> registerDataTreeShard(
+            @Nonnull DOMDataTreeIdentifier prefix, @Nonnull T shard) throws DOMDataTreeShardingConflictException;
 }
index 76999ec5d4e920705fc5ce43034a5a8d41b5c2d6..236e925302f445006fe06a442885b30f33ac3764 100644 (file)
@@ -14,6 +14,7 @@ import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 
 /**
  * A transaction that provides mutation capabilities on a data tree.
+ *
  * <p>
  * For more information on usage and examples, please see the documentation in {@link AsyncWriteTransaction}.
  */
@@ -25,8 +26,10 @@ public interface DOMDataWriteTransaction extends AsyncWriteTransaction<YangInsta
     /**
      * Stores a piece of data at the specified path. This acts as an add / replace
      * operation, which is to say that whole subtree will be replaced by the specified data.
+     *
      * <p>
      * For more information on usage and examples, please see the documentation in {@link AsyncWriteTransaction}.
+     *
      * <p>
      * If you need to make sure that a parent object exists but you do not want modify
      * its pre-existing state by using put, consider using {@link #merge} instead.
@@ -46,8 +49,10 @@ public interface DOMDataWriteTransaction extends AsyncWriteTransaction<YangInsta
      * Merges a piece of data with the existing data at a specified path. Any pre-existing data
      * which is not explicitly overwritten will be preserved. This means that if you store a container,
      * its child lists will be merged.
+     *
      * <p>
      * For more information on usage and examples, please see the documentation in {@link AsyncWriteTransaction}.
+     *
      *<p>
      * If you require an explicit replace operation, use {@link #put} instead.
      *
index 1e19e19f0aae3c6455379bb37fbc9dcec4318ff5..6022b184dc285fde4a7119f8f8b0680d5f7cf698 100644 (file)
@@ -11,12 +11,12 @@ package org.opendaylight.controller.md.sal.dom.api;
 import java.util.Date;
 
 /**
- * Generic event interface
+ * Generic event interface.
  */
 public interface DOMEvent {
 
     /**
-     * Get the time of the event occurrence
+     * Get the time of the event occurrence.
      *
      * @return the event time
      */
index 7e9d56e1c858ae6df074b1ae787ae48a0c961ed1..e3dadf74143245cfa9125744f06011789ad42138 100644 (file)
@@ -20,7 +20,8 @@ import javax.annotation.Nonnull;
  * @param <E> Extension type
  */
 @Beta
-public interface DOMExtensibleService<T extends DOMExtensibleService<T, E>, E extends DOMServiceExtension<T, E>> extends DOMService {
+public interface DOMExtensibleService<T extends DOMExtensibleService<T, E>,
+        E extends DOMServiceExtension<T, E>> extends DOMService {
     /**
      * Return a map of currently-supported extensions, along with accessor services
      * which provide access to the specific functionality bound to this service.
index 3a04ee240a9cf25484d24ffedb8791d66ebbcfb4..b679fc0366aaa2d4494ac4a17bbb101e02930a61 100644 (file)
@@ -8,10 +8,9 @@
 
 package org.opendaylight.controller.md.sal.dom.api;
 
+import com.google.common.base.Optional;
 import org.opendaylight.yangtools.concepts.Identifiable;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
-
-import com.google.common.base.Optional;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 
 public interface DOMMountPoint extends Identifiable<YangInstanceIdentifier> {
index 5972273927ca1a45d81b927e6832b01be9c5c0f5..8880ae01710559af23f70bc4cfd9472dd175f065 100644 (file)
@@ -8,6 +8,7 @@
 
 package org.opendaylight.controller.md.sal.dom.api;
 
+import com.google.common.base.Optional;
 import org.opendaylight.controller.sal.core.api.BrokerService;
 import org.opendaylight.controller.sal.core.api.mount.MountProvisionListener;
 import org.opendaylight.yangtools.concepts.ListenerRegistration;
@@ -15,9 +16,6 @@ import org.opendaylight.yangtools.concepts.ObjectRegistration;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 
-import com.google.common.base.Optional;
-
-
 public interface DOMMountPointService extends BrokerService {
 
     Optional<DOMMountPoint> getMountPoint(YangInstanceIdentifier path);
index f80a124a8d08348719a9ab02a79423cc84bb0661..3b6a88fe91de9c94a8d10554b021adafa620ebd0 100644 (file)
@@ -29,7 +29,8 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      * Well-known value indicating that the implementation is currently not
      * able to accept a notification.
      */
-    ListenableFuture<Object> REJECTED = Futures.immediateFailedFuture(new DOMNotificationRejectedException("Unacceptable blocking conditions encountered"));
+    ListenableFuture<Object> REJECTED = Futures.immediateFailedFuture(
+            new DOMNotificationRejectedException("Unacceptable blocking conditions encountered"));
 
     /**
      * Publish a notification. The result of this method is a {@link ListenableFuture}
@@ -38,6 +39,7 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      * and implementations may use it to convey additional information related to the
      * publishing process.
      *
+     * <p>
      * Abstract subclasses can refine the return type as returning a promise of a
      * more specific type, e.g.:
      *
@@ -46,6 +48,7 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      *     ListenableFuture<? extends DeliveryStatus> putNotification(DOMNotification notification);
      * }
      *
+     * <p>
      * Once the Future succeeds, the resulting object can be queried for traits using
      * instanceof, e.g:
      *
@@ -58,10 +61,12 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      *     }
      * }
      *
+     * <p>
      * In case an implementation is running out of resources, it can block the calling
      * thread until enough resources become available to accept the notification for
      * processing, or it is interrupted.
      *
+     * <p>
      * Caution: completion here means that the implementation has completed processing
      *          of the notification. This does not mean that all existing registrants
      *          have seen the notification. Most importantly, the delivery process at
index b57b5053d5821debd3fd5db3f77178d8ae3e4eff..6c03e9b9fb92933fe06e770a1d8143c94099f9b7 100644 (file)
@@ -8,7 +8,6 @@
 package org.opendaylight.controller.md.sal.dom.api;
 
 /**
- * <p>
  * This exception indicates that given notification can not be processed by corresponding mechanism.
  * More info can be provided in message.
  *
index 33f009a91fab7dd2a6af9225d17e84419112d719..158d9e88c6d9f72c55a2194bba5cac77c5fece5b 100644 (file)
@@ -32,7 +32,8 @@ public interface DOMNotificationService extends DOMService, BrokerService {
      *         null or a SchemaPath which does not represent a valid {@link DOMNotification} type.
      * @throws NullPointerException if either of the arguments is null
      */
-    <T extends DOMNotificationListener> ListenerRegistration<T> registerNotificationListener(@Nonnull T listener, @Nonnull Collection<SchemaPath> types);
+    <T extends DOMNotificationListener> ListenerRegistration<T> registerNotificationListener(
+            @Nonnull T listener, @Nonnull Collection<SchemaPath> types);
 
     /**
      * Register a {@link DOMNotificationListener} to receive a set of notifications. As with
@@ -49,5 +50,6 @@ public interface DOMNotificationService extends DOMService, BrokerService {
      * @throws NullPointerException if listener is null
      */
     // FIXME: Java 8: provide a default implementation of this method.
-    <T extends DOMNotificationListener> ListenerRegistration<T> registerNotificationListener(@Nonnull T listener, SchemaPath... types);
+    <T extends DOMNotificationListener> ListenerRegistration<T> registerNotificationListener(
+            @Nonnull T listener, SchemaPath... types);
 }
index fb38a8f18576dfa108c22548aae879e067ff0ba6..5ead6b7593198826eed54ba8a1f0c47f3cdba80d 100644 (file)
@@ -71,7 +71,8 @@ public abstract class DOMRpcIdentifier {
      * @param contextReference Context reference, null means a global RPC identifier.
      * @return A global RPC identifier, guaranteed to be non-null.
      */
-    public static @Nonnull DOMRpcIdentifier create(final @Nonnull SchemaPath type, final @Nullable YangInstanceIdentifier contextReference) {
+    public static @Nonnull DOMRpcIdentifier create(final @Nonnull SchemaPath type,
+            final @Nullable YangInstanceIdentifier contextReference) {
         if (contextReference == null || contextReference.isEmpty()) {
             return new Global(type);
         }
@@ -120,6 +121,7 @@ public abstract class DOMRpcIdentifier {
 
     @Override
     public final String toString() {
-        return MoreObjects.toStringHelper(this).omitNullValues().add("type", type).add("contextReference", getContextReference()).toString();
+        return MoreObjects.toStringHelper(this).omitNullValues().add("type", type).add("contextReference",
+                getContextReference()).toString();
     }
 }
index cca9a452b8f1fc29a9ad98aba9055921e89e557b..71685285d2f3ead0e6bf881cc553f1a02e979802 100644 (file)
@@ -20,7 +20,8 @@ public class DOMRpcImplementationNotAvailableException extends DOMRpcException {
         super(String.format(format, args));
     }
 
-    public DOMRpcImplementationNotAvailableException(@Nonnull final Throwable cause, @Nonnull final String format, final Object... args) {
+    public DOMRpcImplementationNotAvailableException(@Nonnull final Throwable cause, @Nonnull final String format,
+            final Object... args) {
         super(String.format(format, args), Preconditions.checkNotNull(cause));
     }
 }
index 2d2fa48c058a4513a49f5504a70781f35695cb8e..446960ca058fea3aab958f4e10594ad1d9fe8012 100644 (file)
@@ -14,6 +14,7 @@ import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 /**
  * A chain of DOM Data transactions.
  *
+ * <p>
  * Transactions in a chain need to be committed in sequence and each
  * transaction should see the effects of previous transactions as if they happened. A chain
  * makes no guarantees of atomicity, in fact transactions are committed as soon as possible.
@@ -31,5 +32,4 @@ public interface DOMTransactionChain extends TransactionChain<YangInstanceIdenti
 
     @Override
     DOMDataWriteTransaction newWriteOnlyTransaction();
-
 }
index 12c26a98564ee35a8594e01edeb58bc4cdcf351a..6465e1deb20e248fd090b9f5dfcd8f81945ae81f 100644 (file)
@@ -22,26 +22,27 @@ public abstract class AbstractConsumer implements Consumer, BundleActivator,Serv
     private Broker broker;
 
     @Override
-    public final void start(final BundleContext context) throws Exception {
-        this.context = context;
-        this.startImpl(context);
-        tracker = new ServiceTracker<>(context, Broker.class, this);
+    public final void start(final BundleContext bundleContext) throws Exception {
+        this.context = bundleContext;
+        this.startImpl(bundleContext);
+        tracker = new ServiceTracker<>(bundleContext, Broker.class, this);
         tracker.open();
     }
 
 
 
     @Override
-    public final void stop(final BundleContext context) throws Exception {
-        stopImpl(context);
+    public final void stop(final BundleContext bundleContext) throws Exception {
+        stopImpl(bundleContext);
         broker = null;
         tracker.close();
     }
 
-    protected void startImpl(final BundleContext context) {
+    protected void startImpl(final BundleContext bundleContext) {
         // NOOP
     }
-    protected void stopImpl(final BundleContext context) {
+
+    protected void stopImpl(final BundleContext bundleContext) {
         // NOOP
     }
 
@@ -53,7 +54,7 @@ public abstract class AbstractConsumer implements Consumer, BundleActivator,Serv
 
     @Override
     public Broker addingService(final ServiceReference<Broker> reference) {
-        if(broker == null) {
+        if (broker == null) {
             broker = context.getService(reference);
             broker.registerConsumer(this, context);
             return broker;
index e300279d7b431ef02bbbed600852631c349df234..5cdc26d35e63eeb83e238c2e2c7fe13f6a9e4847 100644 (file)
@@ -27,31 +27,32 @@ public abstract class AbstractProvider implements BundleActivator, Provider,Serv
     }
 
     @Override
-    public final void start(final BundleContext context) throws Exception {
-        this.context = context;
-        this.startImpl(context);
-        tracker = new ServiceTracker<>(context, Broker.class, this);
+    public final void start(final BundleContext bundleContext) throws Exception {
+        this.context = bundleContext;
+        this.startImpl(bundleContext);
+        tracker = new ServiceTracker<>(bundleContext, Broker.class, this);
         tracker.open();
     }
 
-    protected void startImpl(final BundleContext context) {
+    protected void startImpl(final BundleContext bundleContext) {
         // NOOP
     }
-    protected void stopImpl(final BundleContext context) {
+
+    protected void stopImpl(final BundleContext bundleContext) {
         // NOOP
     }
 
     @Override
-    public final void stop(final BundleContext context) throws Exception {
+    public final void stop(final BundleContext bundleContext) throws Exception {
         broker = null;
         tracker.close();
         tracker = null;
-        stopImpl(context);
+        stopImpl(bundleContext);
     }
 
     @Override
     public Broker addingService(final ServiceReference<Broker> reference) {
-        if(broker == null) {
+        if (broker == null) {
             broker = context.getService(reference);
             broker.registerProvider(this, context);
             return broker;
@@ -69,5 +70,4 @@ public abstract class AbstractProvider implements BundleActivator, Provider,Serv
     public void removedService(final ServiceReference<Broker> reference, final Broker service) {
         stopImpl(context);
     }
-
 }
index 01f78cd5aab6bd006acacd29496234edde526508..680a24c2543dd53439ed096051e082ab29ada1e1 100644 (file)
@@ -12,25 +12,25 @@ import org.osgi.framework.BundleContext;
 /**
  * Core component of the SAL layer responsible for wiring the SAL consumers.
  *
+ * <p>
  * The responsibility of the broker is to maintain registration of SAL
  * functionality {@link Consumer}s and {@link Provider}s, store provider and
  * consumer specific context and functionality registration via
  * {@link ConsumerSession} and provide access to infrastructure services, which
  * removes direct dependencies between providers and consumers.
  *
- *
+ * <p>
  * The services are exposed via session.
  *
  * <h3>Session-based access</h3>
  *
+ * <p>
  * The providers and consumers needs to register in order to use the
  * binding-independent SAL layer and to expose functionality via SAL layer.
  *
+ * <p>
  * For more information about session-based access see {@link ConsumerSession}
  * and {@link ProviderSession}
- *
- *
- *
  */
 public interface Broker {
 
@@ -77,6 +77,7 @@ public interface Broker {
      * from consumer, using the {@link Provider#getProviderFunctionality()}, and
      * register that functionality into system and concrete infrastructure
      * services.
+     *
      * <p>
      * The consumer is <b>required to use</b> returned session for all
      * communication with broker or one of the broker services. The session is
@@ -121,8 +122,7 @@ public interface Broker {
         boolean isClosed();
 
         /**
-         * Returns a session specific instance (implementation) of requested
-         * service
+         * Returns a session specific instance (implementation) of requested service.
          *
          * @param service
          *            Broker service
index 7c00c9c2bbae2a1c229ac79d4b483c33f6327238..77622d0e7ade86fdbf980699ef0cb1d1885701e8 100644 (file)
@@ -8,7 +8,6 @@
 package org.opendaylight.controller.sal.core.api;
 
 /**
- *
  * Session-specific instance of the broker functionality.
  *
  * <p>
@@ -19,8 +18,8 @@ package org.opendaylight.controller.sal.core.api;
  *
  * <p>
  * The consumer's (or provider's) instance of specific service could be obtained
- * by invoking {@link org.opendaylight.controller.sal.core.api.Broker.ConsumerSession#getService(Class)} method on session
- * assigned to the consumer.
+ * by invoking {@link org.opendaylight.controller.sal.core.api.Broker.ConsumerSession#getService(Class)}
+ * method on session assigned to the consumer.
  *
  * <p>
  * {@link BrokerService} and {@link Provider} may seem similar, but provider
index b5c39779ab6782566327368d0915a2ec3ca32d6c..b9ac87ae1918bed202469899907057a0f11e68c2 100644 (file)
@@ -8,28 +8,27 @@
 package org.opendaylight.controller.sal.core.api;
 
 import java.util.Collection;
-
 import org.opendaylight.controller.sal.core.api.Broker.ConsumerSession;
 
 /**
- *
  * Defines the component of controller and supplies additional metadata. A
  * component of the controller or application supplies a concrete implementation
  * of this interface.
  *
+ * <p>
  * A user-implemented component (application) which facilitates the SAL and SAL
  * services to access infrastructure services or providers' functionality.
- *
- *
  */
 public interface Consumer {
 
     /**
      * Callback signaling initialization of the consumer session to the SAL.
      *
+     * <p>
      * The consumer MUST use the session for all communication with SAL or
      * retrieving SAL infrastructure services.
      *
+     * <p>
      * This method is invoked by {@link Broker#registerConsumer(Consumer)}
      *
      * @param session
@@ -38,19 +37,21 @@ public interface Consumer {
     void onSessionInitiated(ConsumerSession session);
 
     /**
-     * @deprecated - no longer used or needed
-     * *
-     * Suggested implementation until removed:
-     * {@code
-     * public Collection<ConsumerFunctionality> getConsumerFunctionality() {
-     *    return Collections.emptySet();
-     * }
-     * }
+     * Deprecated.
+     *
+     * @deprecated - no longer used or needed. Suggested implementation until removed:
+     *   {@code
+     *       public Collection<ConsumerFunctionality> getConsumerFunctionality() {
+     *           return Collections.emptySet();
+     *       }
+     *   }
      */
     @Deprecated
     Collection<ConsumerFunctionality> getConsumerFunctionality();
 
     /**
+     * Deprecated.
+     *
      * @deprecated - no longer used or needed
      */
     @Deprecated
index ea6f69c331d1e2a8d3c9d3b0bc230786be2e97d3..e6ba1cc76e493a6a49ada81ca63d895a3992cf92 100644 (file)
@@ -8,11 +8,9 @@
 package org.opendaylight.controller.sal.core.api;
 
 import java.util.Collection;
-
 import org.opendaylight.controller.sal.core.api.Broker.ProviderSession;
 
 /**
- *
  * Defines the component of controller and supplies additional metadata. A
  * component of the controller or application supplies a concrete implementation
  * of this interface.
@@ -21,8 +19,6 @@ import org.opendaylight.controller.sal.core.api.Broker.ProviderSession;
  * A user-implemented component (application) which facilitates the SAL and SAL
  * services to access infrastructure services and to provide functionality to
  * {@link Consumer}s and other providers.
- *
- *
  */
 public interface Provider {
 
@@ -42,9 +38,9 @@ public interface Provider {
     void onSessionInitiated(ProviderSession session);
 
     /**
-     * @deprecated - No longer used or needed
+     * Deprecated.
      *
-     * Suggested implementation until removed:
+     * @deprecated - No longer used or needed. Suggested implementation until removed:
      * {@code
      * public Collection<ProviderFunctionality> getProviderFunctionality() {
      *  return Collections.emptySet();
@@ -55,6 +51,8 @@ public interface Provider {
     Collection<ProviderFunctionality> getProviderFunctionality();
 
     /**
+     * Deprecated.
+     *
      * @deprecated - no longer used or needed
      */
     @Deprecated
index ff3bb890924bfa06157a6e8cceb28173d98bb344..a847d326a32aeb999b509c17db286856c0c2dfcc 100644 (file)
@@ -8,15 +8,10 @@
 package org.opendaylight.controller.sal.core.api;
 
 import java.io.Serializable;
-
 import org.opendaylight.yangtools.concepts.Immutable;
 import org.opendaylight.yangtools.yang.common.QName;
 
-public class RpcRoutingContext implements Immutable, Serializable {
-
-    /**
-     *
-     */
+public final class RpcRoutingContext implements Immutable, Serializable {
     private static final long serialVersionUID = -9079324728075883325L;
 
     private final QName context;
@@ -28,7 +23,7 @@ public class RpcRoutingContext implements Immutable, Serializable {
         this.rpc = rpc;
     }
 
-    public static final RpcRoutingContext create(QName context, QName rpc) {
+    public static RpcRoutingContext create(QName context, QName rpc) {
         return new RpcRoutingContext(context, rpc);
     }
 
@@ -49,30 +44,37 @@ public class RpcRoutingContext implements Immutable, Serializable {
     public int hashCode() {
         final int prime = 31;
         int result = 1;
-        result = prime * result + ((context == null) ? 0 : context.hashCode());
-        result = prime * result + ((rpc == null) ? 0 : rpc.hashCode());
+        result = prime * result + (context == null ? 0 : context.hashCode());
+        result = prime * result + (rpc == null ? 0 : rpc.hashCode());
         return result;
     }
 
     @Override
     public boolean equals(Object obj) {
-        if (this == obj)
+        if (this == obj) {
             return true;
-        if (obj == null)
+        }
+        if (obj == null) {
             return false;
-        if (getClass() != obj.getClass())
+        }
+        if (getClass() != obj.getClass()) {
             return false;
+        }
         RpcRoutingContext other = (RpcRoutingContext) obj;
         if (context == null) {
-            if (other.context != null)
+            if (other.context != null) {
                 return false;
-        } else if (!context.equals(other.context))
+            }
+        } else if (!context.equals(other.context)) {
             return false;
+        }
         if (rpc == null) {
-            if (other.rpc != null)
+            if (other.rpc != null) {
                 return false;
-        } else if (!rpc.equals(other.rpc))
+            }
+        } else if (!rpc.equals(other.rpc)) {
             return false;
+        }
         return true;
     }
 }
index a56a468e20334d91d8c93750c048d7504a9a1db5..673849d09d84c53bad95f1b02d4391d91947c05a 100644 (file)
@@ -15,29 +15,31 @@ import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 import org.opendaylight.yangtools.yang.model.api.SchemaContextListener;
 
 /**
+ * Deprecated.
+ *
  * @deprecated Use {@link DOMSchemaService} instead.
  */
 @Deprecated
 public interface SchemaService extends BrokerService, DOMSchemaService {
 
     /**
-     * Registers a YANG module to session and global context
+     * Registers a YANG module to session and global context.
      */
     void addModule(Module module);
 
     /**
-     * Unregisters a YANG module from session context
+     * Unregisters a YANG module from session context.
      */
     void removeModule(Module module);
 
     /**
-     * Returns session specific YANG schema context
+     * Returns session specific YANG schema context.
      */
     @Override
     SchemaContext getSessionContext();
 
     /**
-     * Returns global schema context
+     * Returns global schema context.
      */
     @Override
     SchemaContext getGlobalContext();
index d2872e256c6b42cdb277153314aa764f915da5c2..c48b4b89ba9076e171d1636888bbb0da705eee08 100644 (file)
@@ -27,12 +27,14 @@ import org.opendaylight.yangtools.yang.data.impl.schema.builder.impl.ImmutableCo
  * can be used.
  */
 public abstract class AbstractDOMDataTreeServiceTestSuite {
-    protected static final QNameModule TEST_MODULE = QNameModule.create(URI.create("urn:opendaylight:params:xml:ns:yang:controller:md:sal:test:store"));
+    protected static final QNameModule TEST_MODULE = QNameModule.create(URI.create(
+            "urn:opendaylight:params:xml:ns:yang:controller:md:sal:test:store"));
 
     protected static final YangInstanceIdentifier UNORDERED_CONTAINER_IID = YangInstanceIdentifier.create(
         new NodeIdentifier(QName.create(TEST_MODULE, "lists")),
         new NodeIdentifier(QName.create(TEST_MODULE, "unordered-container")));
-    protected static final DOMDataTreeIdentifier UNORDERED_CONTAINER_TREE = new DOMDataTreeIdentifier(LogicalDatastoreType.OPERATIONAL, UNORDERED_CONTAINER_IID);
+    protected static final DOMDataTreeIdentifier UNORDERED_CONTAINER_TREE =
+            new DOMDataTreeIdentifier(LogicalDatastoreType.OPERATIONAL, UNORDERED_CONTAINER_IID);
 
     /**
      * Return a reference to the service used in this test. The instance
@@ -46,19 +48,18 @@ public abstract class AbstractDOMDataTreeServiceTestSuite {
     /**
      * A simple unbound producer. It write some basic things into the data store based on the
      * test model.
-     * @throws DOMDataTreeProducerException
-     * @throws TransactionCommitFailedException
      */
     @Test
     public final void testBasicProducer() throws DOMDataTreeProducerException, TransactionCommitFailedException {
         // Create a producer. It is an AutoCloseable resource, hence the try-with pattern
-        try (final DOMDataTreeProducer prod = service().createProducer(Collections.singleton(UNORDERED_CONTAINER_TREE))) {
+        try (DOMDataTreeProducer prod = service().createProducer(Collections.singleton(UNORDERED_CONTAINER_TREE))) {
             assertNotNull(prod);
 
             final DOMDataWriteTransaction tx = prod.createTransaction(true);
             assertNotNull(tx);
 
-            tx.put(LogicalDatastoreType.OPERATIONAL, UNORDERED_CONTAINER_IID, ImmutableContainerNodeBuilder.create().build());
+            tx.put(LogicalDatastoreType.OPERATIONAL, UNORDERED_CONTAINER_IID,
+                    ImmutableContainerNodeBuilder.create().build());
 
             final CheckedFuture<Void, TransactionCommitFailedException> f = tx.submit();
             assertNotNull(f);
index af7a32924b783ebfafa1f43a2726b9a635387270..d1bd2efe209853fedb1dc7b8314982bfa3c5cac7 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.md.sal.test.model.util;
 
 import com.google.common.collect.ImmutableList;
+import java.util.Arrays;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.controller.md.sal.test.augment.rev140709.TreeComplexUsesAugment;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.controller.md.sal.test.augment.rev140709.TreeComplexUsesAugmentBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.controller.md.sal.test.augment.rev140709.TreeLeafOnlyUsesAugment;
@@ -26,9 +27,7 @@ import org.opendaylight.yangtools.yang.binding.Augmentation;
 import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 
-import java.util.Arrays;
-
-public class ListsBindingUtils {
+public final class ListsBindingUtils {
 
     private static final InstanceIdentifier<Top> TOP_PATH = InstanceIdentifier.create(Top.class);
 
@@ -54,7 +53,8 @@ public class ListsBindingUtils {
         return path(top).augmentation(TreeComplexUsesAugment.class).child(ListViaUses.class, uses);
     }
 
-    public static <T extends DataObject & Augmentation<TopLevelList>> InstanceIdentifier<T> path(final TopLevelListKey key, final Class<T> augmentation) {
+    public static <T extends DataObject & Augmentation<TopLevelList>> InstanceIdentifier<T> path(
+            final TopLevelListKey key, final Class<T> augmentation) {
         return path(key).augmentation(augmentation);
     }
 
@@ -73,7 +73,7 @@ public class ListsBindingUtils {
     }
 
     public static TreeComplexUsesAugment complexUsesAugment(final ListViaUsesKey... keys) {
-        ImmutableList.Builder<ListViaUses> listViaUses = ImmutableList.<ListViaUses> builder();
+        ImmutableList.Builder<ListViaUses> listViaUses = ImmutableList.<ListViaUses>builder();
         for (ListViaUsesKey key : keys) {
             listViaUses.add(new ListViaUsesBuilder().setKey(key).build());
         }