hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@duboce.net
Subject Re: Review Request: hbase-2998 rolling-restart.sh shouldn't rely on zoo.cfg
Date Thu, 21 Oct 2010 08:54:34 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-10-21 01:54:34.658192)

Review request for hbase, Jean-Daniel Cryans and Jonathan Gray.


New patch includes faster assign of regions on startup (Uses async create/exists-set-watcher).
 Getting this working helps w/ rolling restart tests.  Assign and watcher set for 2k regions
runs fast now... used to be 90 seconds for 2k regions over 10 servers ... now its a matter
of seconds for total bulk assign of all regions in just over a minute.

This patch is not yet ready.  I need to test more.


Fix 'hbase zkcli' so it reads zk ensemble location from hbase config/zoo.cfg.  This fixes
rolling restart.  Patch also includes fix so rolling restarts work on new master.

A src/test/java/org/apache/hadoop/hbase/zookeeper/TestZooKeeperMainServerArg.java
  Test for new TZMSA class.
M src/main/java/org/apache/hadoop/hbase/zookeeper/ZKServerTool.java
  Minor edit of javadoc.
A src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperMainServerArg.java
  Tool to emit what ZooKeeperMain wants for a server argument.
M src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
  (isAbort): Added.
M src/main/java/org/apache/hadoop/hbase/regionserver/ShutdownHook.java
  Shutdown hook now needs to startup region shutdowns since  new
  master changed how shutdown sequence runs.
M src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java
  Don't do opens if server is stopped.
M src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java
  Minor formatting.
M bin/hbase
  Run new ZKMSA tool to figure '-server host:port' to pass ZKM
M bin/hbase-daemon.sh
  Make default wait be longer.

This addresses bug hbase-2998.

Diffs (updated)

  trunk/bin/hbase 1025815 
  trunk/bin/hbase-daemon.sh 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/catalog/CatalogTracker.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/executor/ExecutorService.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java 1025815

  trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKAssign.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKServerTool.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java 1025815 
  trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWatcher.java 1025815 

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




View raw message