You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ga...@apache.org on 2008/03/26 06:19:03 UTC

svn commit: r641154 - /geronimo/sandbox/concurrent/geronimo-concurrent-core/src/main/java/org/apache/geronimo/concurrent/executor/ManagedExecutorServiceFacade.java

Author: gawor
Date: Tue Mar 25 22:19:02 2008
New Revision: 641154

URL: http://svn.apache.org/viewvc?rev=641154&view=rev
Log:
doh! throw IllegalStateException instead of IllegalArgumentException

Modified:
    geronimo/sandbox/concurrent/geronimo-concurrent-core/src/main/java/org/apache/geronimo/concurrent/executor/ManagedExecutorServiceFacade.java

Modified: geronimo/sandbox/concurrent/geronimo-concurrent-core/src/main/java/org/apache/geronimo/concurrent/executor/ManagedExecutorServiceFacade.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/concurrent/geronimo-concurrent-core/src/main/java/org/apache/geronimo/concurrent/executor/ManagedExecutorServiceFacade.java?rev=641154&r1=641153&r2=641154&view=diff
==============================================================================
--- geronimo/sandbox/concurrent/geronimo-concurrent-core/src/main/java/org/apache/geronimo/concurrent/executor/ManagedExecutorServiceFacade.java (original)
+++ geronimo/sandbox/concurrent/geronimo-concurrent-core/src/main/java/org/apache/geronimo/concurrent/executor/ManagedExecutorServiceFacade.java Tue Mar 25 22:19:02 2008
@@ -134,7 +134,7 @@
         
     public boolean awaitTermination(long timeout, TimeUnit unit) throws InterruptedException {
         if (this.serverManaged) {
-            throw new IllegalArgumentException();
+            throw new IllegalStateException();
         } else {
             return this.executor.awaitTermination(timeout, unit);
         }
@@ -142,7 +142,7 @@
     
     public boolean isShutdown() {
         if (this.serverManaged) {
-            throw new IllegalArgumentException();
+            throw new IllegalStateException();
         } else {
             return this.executor.isShutdown();
         }
@@ -150,7 +150,7 @@
 
     public boolean isTerminated() {
         if (this.serverManaged) {
-            throw new IllegalArgumentException();
+            throw new IllegalStateException();
         } else {
             return this.executor.isTerminated();
         }
@@ -158,7 +158,7 @@
 
     public void shutdown() {
         if (this.serverManaged) {
-            throw new IllegalArgumentException();
+            throw new IllegalStateException();
         } else {
             this.executor.shutdown();
         }
@@ -166,7 +166,7 @@
 
     public List<Runnable> shutdownNow() {
         if (this.serverManaged) {
-            throw new IllegalArgumentException();
+            throw new IllegalStateException();
         } else {
             return this.executor.shutdownNow();
         }