[jboss-cvs] JBossCache/src/org/jboss/cache/demo ...

Galder Zamarreno galder.zamarreno at jboss.com
Sun May 20 06:51:50 EDT 2007


  User: gzamarreno
  Date: 07/05/20 06:51:50

  Modified:    src/org/jboss/cache/demo  JBossCacheGUI.java
  Log:
  [JBCACHE-1041] use newCachedThreadPool instead of fixed thread pool.
  
  Revision  Changes    Path
  1.3       +4 -4      JBossCache/src/org/jboss/cache/demo/JBossCacheGUI.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: JBossCacheGUI.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/demo/JBossCacheGUI.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -b -r1.2 -r1.3
  --- JBossCacheGUI.java	18 May 2007 19:38:28 -0000	1.2
  +++ JBossCacheGUI.java	20 May 2007 10:51:50 -0000	1.3
  @@ -69,11 +69,11 @@
      private boolean useConsole = false;
   
      /**
  -    * Run any work that happens in this interface in a separate thread.  This is good practise. Initialised to 3
  -    * threads based on the number of concurrent actions that could be happening at the same time (view change,
  -    * click on node,...etc) 
  +    * Run any work that happens in this interface in a separate thread.  This is good practise. Unless there's a
  +    * resource-managemenet requirement, such as in an application server, newCachedThreadPool is a good option as it
  +    * provides better queuing performance.
       */
  -   private Executor executor = Executors.newFixedThreadPool(3);
  +   private Executor executor = Executors.newCachedThreadPool();
   
      public JBossCacheGUI(CacheModelDelegate cacheDelegate, boolean useConsole) throws Exception
      {
  
  
  



More information about the jboss-cvs-commits mailing list