svn commit: r1181966 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HTable.java

[ns...@apache.org]

Author: nspiegelberg
Date: Tue Oct 11 17:44:38 2011
New Revision: 1181966

URL: http://svn.apache.org/viewvc?rev=1181966&view=rev
Log:
Make corePool as a configurable parameter in HTable

Summary: As discussed with Charles, we tried to make the corePool as a
configurable parameter in HTable. So we can tune this parameter in the config
file.

Test Plan:
run all the unit tests.
will test in the ods client side.

Reviewers: kannan, kranganathan, vinodv, cgthayer, mbautin

Reviewed By: kannan

CC: , hbase@lists, kannan

Differential Revision: 317796

Modified:
    hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HTable.java

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HTable.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HTable.java?rev=1181966&r1=1181965&r2=1181966&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HTable.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HTable.java Tue Oct 11 17:44:38 2011
@@ -152,10 +152,10 @@ public class HTable implements HTableInt
       nrHRS = 10;
     }
     int nrThreads = conf.getInt("hbase.htable.threads.max", nrHRS);
-
+    int corePools = conf.getInt("hbase.htable.threads.corePool", 0);
     // Unfortunately Executors.newCachedThreadPool does not allow us to
     // set the maximum size of the pool, so we have to do it ourselves.
-    this.pool = new ThreadPoolExecutor(0, nrThreads,
+    this.pool = new ThreadPoolExecutor(corePools, nrThreads,
         60, TimeUnit.SECONDS,
         new LinkedBlockingQueue<Runnable>(),
         new DaemonThreadFactory());