----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5077/#review7772 -----------------------------------------------------------
Ship it! +1, thanks Sebastian! I'm assuming it also passes 'mvn verify', but please double check before you commit. - Avery On 2012-05-10 09:32:10, Sebastian Schelter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5077/ > ----------------------------------------------------------- > > (Updated 2012-05-10 09:32:10) > > > Review request for giraph. > > > Summary > ------- > > All temporary files that the tests generate are now written to > /tmp/_giraphTests including zooKeeper files, checkpoints etc. > > This behavior will be automatically configured whenever > InternalVertexRunner.run() or BspCase.prepareJob() is used. > > Usually I can't stop myself once I have my refactoring hat on, therefore I > also tidied up a lot of minor stuff, removed code duplications etc. > > > This addresses bug GIRAPH-20. > https://issues.apache.org/jira/browse/GIRAPH-20 > > > Diffs > ----- > > trunk/src/main/java/org/apache/giraph/examples/SimplePageRankVertex.java > 1336504 > trunk/src/main/java/org/apache/giraph/graph/GraphMapper.java 1336504 > trunk/src/main/java/org/apache/giraph/graph/TextAggregatorWriter.java > 1336504 > trunk/src/main/java/org/apache/giraph/utils/FileUtils.java PRE-CREATION > trunk/src/main/java/org/apache/giraph/utils/InternalVertexRunner.java > 1336504 > trunk/src/test/java/org/apache/giraph/BspCase.java 1336504 > trunk/src/test/java/org/apache/giraph/TestAutoCheckpoint.java 1336506 > trunk/src/test/java/org/apache/giraph/TestBspBasic.java 1336504 > trunk/src/test/java/org/apache/giraph/TestGraphPartitioner.java 1336504 > trunk/src/test/java/org/apache/giraph/TestJsonBase64Format.java 1336504 > trunk/src/test/java/org/apache/giraph/TestManualCheckpoint.java 1336506 > trunk/src/test/java/org/apache/giraph/TestMutateGraphVertex.java 1336504 > trunk/src/test/java/org/apache/giraph/TestNotEnoughMapTasks.java 1336504 > trunk/src/test/java/org/apache/giraph/TestZooKeeperExt.java 1336504 > trunk/src/test/java/org/apache/giraph/graph/TestEdgeListVertex.java 1336504 > > Diff: https://reviews.apache.org/r/5077/diff > > > Testing > ------- > > successfully passed local and pseudo-distributed tests with Hadoop 0.20.203 > > > Thanks, > > Sebastian > >