----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1064/#review1632 -----------------------------------------------------------
Ship it! Looks good to me. Nice test :) - Jonathan On 2010-10-22 23:27:28, Jonathan Gray wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.cloudera.org/r/1064/ > ----------------------------------------------------------- > > (Updated 2010-10-22 23:27:28) > > > Review request for hbase and stack. > > > Summary > ------- > > See HBASE-3139 > > > This addresses bug hbase-3139. > http://issues.apache.org/jira/browse/hbase-3139 > > > Diffs > ----- > > 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 > >