Propagate @Nonnull and @Nullable annotations
[yangtools.git] / common / util / src / main / java / org / opendaylight / yangtools / util / concurrent / AsyncNotifyingListeningExecutorService.java
index ef4670d9c564791eac85ce59277b30a01a825730..b23982e31057bd8e95169b2bb9d4d581161e5d9e 100644 (file)
@@ -8,38 +8,45 @@
 
 package org.opendaylight.yangtools.util.concurrent;
 
+import com.google.common.base.MoreObjects;
+import com.google.common.base.MoreObjects.ToStringHelper;
+import com.google.common.base.Preconditions;
+import com.google.common.util.concurrent.AbstractListeningExecutorService;
+import com.google.common.util.concurrent.ListenableFuture;
 import java.util.List;
 import java.util.concurrent.Callable;
 import java.util.concurrent.Executor;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.TimeUnit;
-
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
-import com.google.common.base.Objects;
-import com.google.common.base.Objects.ToStringHelper;
-import com.google.common.base.Preconditions;
-import com.google.common.util.concurrent.AbstractListeningExecutorService;
-import com.google.common.util.concurrent.ListenableFuture;
-
 /**
- * An {@link ListeningExecutorService} implementation that also allows for an {@link Executor} to be
- * specified on construction that is used to execute {@link ListenableFuture} callback Runnables,
- * registered via {@link Futures#addCallback} or {@link ListenableFuture#addListener} directly,
- * asynchronously when a task that is run on this executor completes. This is useful when you want
- * to guarantee listener callback executions are off-loaded onto another thread to avoid blocking
- * the thread that completed the task, as a common use case is to pass an executor that runs tasks
- * in the same thread as the caller (ie <code>MoreExecutors#sameThreadExecutor</code>}) to
+ * An {@link com.google.common.util.concurrent.ListeningExecutorService}
+ * implementation that also allows for an {@link Executor} to be specified on
+ * construction that is used to execute {@link ListenableFuture} callback
+ * Runnables, registered via
+ * {@link com.google.common.util.concurrent.Futures#addCallback} or
+ * {@link ListenableFuture#addListener} directly, asynchronously when a task
+ * that is run on this executor completes. This is useful when you want to
+ * guarantee listener callback executions are off-loaded onto another thread to
+ * avoid blocking the thread that completed the task, as a common use case is to
+ * pass an executor that runs tasks in the same thread as the caller (ie
+ * <code>MoreExecutors#sameThreadExecutor</code>}) to
  * {@link ListenableFuture#addListener}.
+ *
  * <p>
- * Most commonly, this class would be used in lieu of <code>MoreExecutors#listeningDecorator<code>
- * when the underlying delegate Executor is single-threaded, in which case, you may not want
- * ListenableFuture callbacks to block the single thread.
+ * Most commonly, this class would be used in lieu of
+ * <code>MoreExecutors#listeningDecorator</code> when the underlying delegate
+ * Executor is single-threaded, in which case, you may not want ListenableFuture
+ * callbacks to block the single thread.
+ *
  * <p>
- * Note: the Executor specified on construction does not replace the Executor specified in
- * {@link ListenableFuture#addListener}. The latter Executor is still used however, if it is
- * detected that the listener Runnable would execute in the thread that completed the task, the
- * listener is executed on Executor specified on construction.
+ * Note: the Executor specified on construction does not replace the Executor
+ * specified in {@link ListenableFuture#addListener}. The latter Executor is
+ * still used however, if it is detected that the listener Runnable would
+ * execute in the thread that completed the task, the listener is executed on
+ * Executor specified on construction.
  *
  * @author Thomas Pantelis
  * @see AsyncNotifyingListenableFutureTask
@@ -56,8 +63,8 @@ public class AsyncNotifyingListeningExecutorService extends AbstractListeningExe
      * @param listenableFutureExecutor the executor used to run listener callbacks asynchronously.
      *     If null, no executor is used.
      */
-    public AsyncNotifyingListeningExecutorService( ExecutorService delegate,
-            @Nullable Executor listenableFutureExecutor ) {
+    public AsyncNotifyingListeningExecutorService( final ExecutorService delegate,
+            @Nullable final Executor listenableFutureExecutor ) {
         this.delegate = Preconditions.checkNotNull( delegate );
         this.listenableFutureExecutor = listenableFutureExecutor;
     }
@@ -67,7 +74,7 @@ public class AsyncNotifyingListeningExecutorService extends AbstractListeningExe
      *
      * @param task the Callable to execute
      */
-    private <T> AsyncNotifyingListenableFutureTask<T> newFutureTask( Callable<T> task ) {
+    private <T> AsyncNotifyingListenableFutureTask<T> newFutureTask( final Callable<T> task ) {
         return AsyncNotifyingListenableFutureTask.create( task, listenableFutureExecutor );
     }
 
@@ -76,7 +83,7 @@ public class AsyncNotifyingListeningExecutorService extends AbstractListeningExe
      *
      * @param task the Runnable to execute
      */
-    private <T> AsyncNotifyingListenableFutureTask<T> newFutureTask( Runnable task, T result ) {
+    private <T> AsyncNotifyingListenableFutureTask<T> newFutureTask( final Runnable task, final T result ) {
         return AsyncNotifyingListenableFutureTask.create( task, result, listenableFutureExecutor );
     }
 
@@ -88,7 +95,7 @@ public class AsyncNotifyingListeningExecutorService extends AbstractListeningExe
     }
 
     @Override
-    public boolean awaitTermination( long timeout, TimeUnit unit ) throws InterruptedException {
+    public boolean awaitTermination( final long timeout, @Nonnull final TimeUnit unit ) throws InterruptedException {
         return delegate.awaitTermination( timeout, unit );
     }
 
@@ -107,44 +114,48 @@ public class AsyncNotifyingListeningExecutorService extends AbstractListeningExe
         delegate.shutdown();
     }
 
+    @Nonnull
     @Override
     public List<Runnable> shutdownNow() {
         return delegate.shutdownNow();
     }
 
     @Override
-    public void execute( Runnable command ) {
+    public void execute( @Nonnull final Runnable command ) {
         delegate.execute( command );
     }
 
+    @Nonnull
     @Override
-    public <T> ListenableFuture<T> submit( Callable<T> task ) {
+    public <T> ListenableFuture<T> submit( final Callable<T> task ) {
         AsyncNotifyingListenableFutureTask<T> futureTask = newFutureTask( task );
         delegate.execute( futureTask );
         return futureTask;
     }
 
+    @Nonnull
     @Override
-    public ListenableFuture<?> submit( Runnable task ) {
+    public ListenableFuture<?> submit( final Runnable task ) {
         AsyncNotifyingListenableFutureTask<Void> futureTask = newFutureTask( task, null );
         delegate.execute( futureTask );
         return futureTask;
     }
 
+    @Nonnull
     @Override
-    public <T> ListenableFuture<T> submit( Runnable task, T result ) {
+    public <T> ListenableFuture<T> submit( final Runnable task, final T result ) {
         AsyncNotifyingListenableFutureTask<T> futureTask = newFutureTask( task, result );
         delegate.execute( futureTask );
         return futureTask;
     }
 
-    protected ToStringHelper addToStringAttributes( ToStringHelper toStringHelper ) {
+    protected ToStringHelper addToStringAttributes( final ToStringHelper toStringHelper ) {
         return toStringHelper;
     }
 
     @Override
-    public final String toString(){
-        return addToStringAttributes( Objects.toStringHelper( this )
+    public final String toString() {
+        return addToStringAttributes( MoreObjects.toStringHelper( this )
                 .add( "delegate", delegate ) ).toString();
     }
 }