HBASE-14433 Set down the client executor core thread count from 256 t…

…o number of processors
huasir · Sep 16, 2015 · d2e3381 · d2e3381
1 parent 27a993d
commit d2e3381
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionImplementation.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionImplementation.java
@@ -370,7 +370,9 @@ private ExecutorService getBatchPool() {
       synchronized (this) {
         if (batchPool == null) {
           this.batchPool = getThreadPool(conf.getInt("hbase.hconnection.threads.max", 256),
-              conf.getInt("hbase.hconnection.threads.core", 256), "-shared-", null); 
+            conf.getInt("hbase.hconnection.threads.core",
+              Runtime.getRuntime().availableProcessors()),
+            "-shared-", null); 
           this.cleanupPool = true;
         }
       }
@@ -385,7 +387,7 @@ private ExecutorService getThreadPool(int maxThreads, int coreThreads, String na
       maxThreads = Runtime.getRuntime().availableProcessors() * 8;
     }
     if (coreThreads == 0) {
-      coreThreads = Runtime.getRuntime().availableProcessors() * 8;
+      coreThreads = Runtime.getRuntime().availableProcessors();
     }
     long keepAliveTime = conf.getLong("hbase.hconnection.threads.keepalivetime", 60);
     BlockingQueue<Runnable> workQueue = passedWorkQueue;

diff --git a/hbase-client/src/test/resources/hbase-site.xml b/hbase-client/src/test/resources/hbase-site.xml
@@ -25,4 +25,12 @@
     <name>hbase.defaults.for.version.skip</name>
     <value>true</value>
   </property>
+  <property>
+    <name>hbase.hconnection.meta.lookup.threads.core</name>
+    <value>4</value>
+  </property>
+  <property>
+    <name>hbase.hconnection.threads.keepalivetime</name>
+    <value>3</value>
+  </property>
 </configuration>
diff --git a/hbase-server/src/test/resources/hbase-site.xml b/hbase-server/src/test/resources/hbase-site.xml
@@ -162,4 +162,12 @@
       Enable replay sanity checks on procedure tests.
     </description>
   </property>
+  <property>
+    <name>hbase.hconnection.meta.lookup.threads.core</name>
+    <value>4</value>
+  </property>
+  <property>
+    <name>hbase.hconnection.threads.keepalivetime</name>
+    <value>3</value>
+  </property>
 </configuration>