----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9350/ -----------------------------------------------------------
(Updated Feb. 8, 2013, 1:04 a.m.) Review request for giraph. Changes ------- Sorry about the patch spam. Looking at the RB diff, I forgot to remove something I'm adding to another JIRA. Otherwise, we're good to go on this. Thanks! Description ------- Refactor CLI parsing of args and the assembly of the GiraphConfiguration into its own class so that other future Runner classes besides GiraphRunner can use it w/o duplication. You get the idea. Diffs (updated) ----- giraph-core/src/main/java/org/apache/giraph/GiraphRunner.java b6a6113 giraph-core/src/main/java/org/apache/giraph/job/GiraphTypeValidator.java de2a66d giraph-core/src/main/java/org/apache/giraph/utils/CliParserUtils.java PRE-CREATION Diff: https://reviews.apache.org/r/9350/diff/ Testing ------- Thanks, Eli Reisman
