----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10478/#review19194 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On April 15, 2013, 2:57 p.m., Venkat Ranganathan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10478/ > ----------------------------------------------------------- > > (Updated April 15, 2013, 2:57 p.m.) > > > Review request for Sqoop and Jarek Cecho. > > > Description > ------- > > Recognize --hadoop-home also as backward compatible option > > > Diffs > ----- > > src/java/org/apache/sqoop/tool/BaseSqoopTool.java 684d4a5 > src/test/com/cloudera/sqoop/TestSqoopOptions.java fcc38eb > > Diff: https://reviews.apache.org/r/10478/diff/ > > > Testing > ------- > > Added two tests (one to recognize hadoop-home) and one to test > hadoop-mapred-home overriding hadoop-home) > All tests pass > > > Thanks, > > Venkat Ranganathan > >
