----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1057/#review1624 -----------------------------------------------------------
Ship it! +1 after doing changes we discussed on IRC. Namely to make sure that the shutdown methods are idempotent and will work for stopping a backup master and that TestMasterFailover passes. Also some minor logging/comment changes around deleting root location. - Jonathan On 2010-10-22 11:59:16, stack wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.cloudera.org/r/1057/ > ----------------------------------------------------------- > > (Updated 2010-10-22 11:59:16) > > > Review request for hbase, Jean-Daniel Cryans and Jonathan Gray. > > > Summary > ------- > > 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. > http://issues.apache.org/jira/browse/hbase-2998 > > > Diffs > ----- > > trunk/bin/hbase 1026448 > trunk/bin/hbase-daemon.sh 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/catalog/CatalogTracker.java > 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/executor/ExecutorService.java > 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java > 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1026448 > > trunk/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java > 1026448 > > trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java > 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKAssign.java 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKServerTool.java > 1026448 > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java 1026448 > > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperMainServerArg.java > PRE-CREATION > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWatcher.java > 1026448 > > trunk/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZooKeeperMainServerArg.java > PRE-CREATION > > Diff: http://review.cloudera.org/r/1057/diff > > > Testing > ------- > > > Thanks, > > stack > >
