----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/380/#review474 -----------------------------------------------------------
Ship it! Just one comment trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWrapper.java <http://review.hbase.org/r/380/#comment1965> What is the watcher being triggered if data is not changing? - stack On 2010-07-23 15:08:16, Karthik Ranganathan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.hbase.org/r/380/ > ----------------------------------------------------------- > > (Updated 2010-07-23 15:08:16) > > > Review request for hbase, stack and Kannan Muthukkaruppan. > > > Summary > ------- > > Region permanently offlined - if the ZNode is already in the target state, do > not update it again. > > > This addresses bug HBASE-2866. > http://issues.apache.org/jira/browse/HBASE-2866 > > > Diffs > ----- > > trunk/src/main/java/org/apache/hadoop/hbase/master/ZKUnassignedWatcher.java > 967128 > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWrapper.java > 967128 > > Diff: http://review.hbase.org/r/380/diff > > > Testing > ------- > > Ran unit tests, went through fine (except TestRowAtomicity, which is known to > be failing). > > > Thanks, > > Karthik > >
