hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@duboce.net
Subject Re: Review Request: Unit test and fix for multi-threaded executor service
Date Sat, 23 Oct 2010 06:27:28 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1064/
-----------------------------------------------------------

(Updated 2010-10-22 23:27:28.712219)


Review request for hbase and stack.


Changes
-------

Remove priority queue -- not needed.  Fix naming format for threads.  Make the thread pool
max and corepool size the same so we have a fixed pool size.  Adjust thread pool sizes down.
 They were upped when we weren't getting throughput when we thought executorservice was doing
parallelism but wasnt'.


Summary
-------

See HBASE-3139


This addresses bug hbase-3139.
    http://issues.apache.org/jira/browse/hbase-3139


Diffs (updated)
-----

  trunk/src/main/java/org/apache/hadoop/hbase/executor/ExecutorService.java 1026564 
  trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1026564 
  trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 1026564 
  trunk/src/test/java/org/apache/hadoop/hbase/executor/TestExecutorService.java 1026564 

Diff: http://review.cloudera.org/r/1064/diff


Testing
-------


Thanks,

Jonathan


Mime
View raw message