Author: apico
Date: 2010-09-20 10:58:31 -0700 (Mon, 20 Sep 2010)
New Revision: 21949

Modified:
   
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientImplTunables.java
   
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientManager.java
   
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/util/BridgeRestUtil.java
Log:
changed all .awaitTermination() to 1 second to improve performance. Now "failed 
to..." messages are generated by the code works.  Should refactor to produce 
better error messages without sacrificing performance.

Modified: 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientImplTunables.java
===================================================================
--- 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientImplTunables.java
 2010-09-20 17:05:34 UTC (rev 21948)
+++ 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientImplTunables.java
 2010-09-20 17:58:31 UTC (rev 21949)
@@ -226,7 +226,7 @@
                     }
                 });
 
-                if (executor.awaitTermination(10, TimeUnit.SECONDS)) {
+                if (executor.awaitTermination(1, TimeUnit.SECONDS)) {
                     executor.shutdown();
                 }
 

Modified: 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientManager.java
===================================================================
--- 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientManager.java
      2010-09-20 17:05:34 UTC (rev 21948)
+++ 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/IDMapperClientManager.java
      2010-09-20 17:58:31 UTC (rev 21949)
@@ -110,7 +110,7 @@
 
     private static ExecutorService executor = null;
 
-    private static int waitSeconds = 5;
+    private static int waitSeconds = 1;
 
     static {
 //        new IDMapperClientManager();

Modified: 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/util/BridgeRestUtil.java
===================================================================
--- 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/util/BridgeRestUtil.java
        2010-09-20 17:05:34 UTC (rev 21948)
+++ 
csplugins/trunk/soc/jgao/IDMapping/src/csplugins/id/mapping/util/BridgeRestUtil.java
        2010-09-20 17:58:31 UTC (rev 21949)
@@ -109,7 +109,7 @@
         });
 
         try {
-            if (!executor.awaitTermination(10, TimeUnit.SECONDS)) {
+            if (!executor.awaitTermination(1, TimeUnit.SECONDS)) {
                 System.err.println("Failed to connect to "+strUrl);
                 executor.shutdown();
             }

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to