----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4364/#review6252 -----------------------------------------------------------
Ship it! +1 - Arvind On 2012-03-15 16:24:38, Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4364/ > ----------------------------------------------------------- > > (Updated 2012-03-15 16:24:38) > > > Review request for Sqoop, Arvind Prabhakar and Bilung Lee. > > > Summary > ------- > > I've add parameter validation fragment that is looking for this incorrect > usage and warning user. > > > This addresses bug SQOOP-464. > https://issues.apache.org/jira/browse/SQOOP-464 > > > Diffs > ----- > > /src/java/org/apache/sqoop/tool/BaseSqoopTool.java 1300633 > > Diff: https://reviews.apache.org/r/4364/diff > > > Testing > ------- > > * ant test -Dhadoopversion={20,23,100} > * On live CDH based cluster > > > Thanks, > > Jarek > >
