----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1056/#review1620 -----------------------------------------------------------
Ship it! +1 on getting it in as is. I think its going to kill my bulk assign improvements but yesterday discussing w/ Jon and Karthik, there are plenty of options to explore yet... We'll do in another issue. - stack On 2010-10-20 16:47:05, Jonathan Gray wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.cloudera.org/r/1056/ > ----------------------------------------------------------- > > (Updated 2010-10-20 16:47:05) > > > Review request for hbase, Todd Lipcon and stack. > > > Summary > ------- > > Adds a sync(path) operation into ZKW and three calls into it from the CAS > operations in ZKAssign. > > > This addresses bug HBASE-3136. > http://issues.apache.org/jira/browse/HBASE-3136 > > > Diffs > ----- > > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKAssign.java 1025790 > trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWatcher.java > 1025790 > > Diff: http://review.cloudera.org/r/1056/diff > > > Testing > ------- > > Still need to test more. I'm not sure it's possible (or feasible in a > reasonable amount of time) to make a unit test for this. We'd probably need > to dig into ZK or mock the hell out of stuff. > > > Thanks, > > Jonathan > >
