----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7674/#review12665 -----------------------------------------------------------
Looks good to me, +1. Feels a bit like manually decorating each method, but it's simple anyway. http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/main/java/org/apache/giraph/GiraphConfiguration.java <https://reviews.apache.org/r/7674/#comment27006> zK -> zk - Alessandro Presta On Oct. 20, 2012, 12:19 a.m., Avery Ching wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7674/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2012, 12:19 a.m.) > > > Review request for giraph. > > > Description > ------- > > ZooKeeperExt now uses composition to ensure that all operations can handle > ConnectionLossException. > > > This addresses bug GIRAPH-382. > https://issues.apache.org/jira/browse/GIRAPH-382 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/main/java/org/apache/giraph/GiraphConfiguration.java > 1399984 > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/BspService.java > 1399984 > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/BspServiceMaster.java > 1399984 > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/BspServiceWorker.java > 1399984 > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/main/java/org/apache/giraph/zk/ZooKeeperExt.java > 1399984 > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/test/java/org/apache/giraph/BspCase.java > 1399984 > > http://svn.apache.org/repos/asf/giraph/trunk/giraph/src/test/java/org/apache/giraph/TestZooKeeperExt.java > 1399984 > > Diff: https://reviews.apache.org/r/7674/diff/ > > > Testing > ------- > > Unittests. > > > Thanks, > > Avery Ching > >
