----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2874/#review3334 -----------------------------------------------------------
Ship it! +1 Thanks for the patch Jarcec. Please attach the patch to the JIRA for commit. - Arvind On 2011-11-17 18:47:44, Jarek Jarcec wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2874/ > ----------------------------------------------------------- > > (Updated 2011-11-17 18:47:44) > > > Review request for Sqoop and Arvind Prabhakar. > > > Summary > ------- > > I've moved the functionality like we did with other classes. > > > This addresses bug SQOOP-393. > https://issues.apache.org/jira/browse/SQOOP-393 > > > Diffs > ----- > > /src/java/com/cloudera/sqoop/SqoopOptions.java 1201319 > /src/java/org/apache/sqoop/SqoopOptions.java 1201319 > > Diff: https://reviews.apache.org/r/2874/diff > > > Testing > ------- > > I've passed basic tests. > > > Thanks, > > Jarek > >
