[jboss-cvs] JBossAS SVN: r90884 - projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins.

[jboss-cvs-commits at lists.jboss.org]

Author: kabir.khan at jboss.com
Date: 2009-07-07 05:00:56 -0400 (Tue, 07 Jul 2009)
New Revision: 90884

Modified:
   projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
Log:
[JBKERNEL-25] Better comments for executor

Modified: projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
===================================================================
--- projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2009-07-07 08:36:38 UTC (rev 90883)
+++ projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2009-07-07 09:00:56 UTC (rev 90884)
@@ -67,7 +67,9 @@
    /** The lock */
    private ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
 
-   /** The executor */
+   /** The executor used to install ASYNCHRONOUS contexts. It must have a saturation policy of
+    * (or semantically similar to) ThreadPoolExecutor.AbortPolicy or ThreadPoolExecutor.CallerRunsPolicy 
+    */
    private Executor executor;
 
    /** Whether we are shutdown */
@@ -129,7 +131,8 @@
    }
 
    /**
-    * Set the executor.
+    * Set the executor used to install ASYNCHRONOUS contexts. It must have a saturation policy of
+    * (or semantically similar to) ThreadPoolExecutor.AbortPolicy or ThreadPoolExecutor.CallerRunsPolicy.
     *
     * @param executor the executor
     */