----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2270/#review2456 -----------------------------------------------------------
Ship it! - Bilung On 2011-10-07 23:04:25, Arvind Prabhakar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2270/ > ----------------------------------------------------------- > > (Updated 2011-10-07 23:04:25) > > > Review request for Sqoop and Bilung Lee. > > > Summary > ------- > > Modified TestBoundaryQuery to introduce a test that reproduces this problem. > Also added a validation exception that is raised when the user uses a > boundary query without specifying a split-by column. > > > This addresses bug SQOOP-359. > https://issues.apache.org/jira/browse/SQOOP-359 > > > Diffs > ----- > > /src/java/com/cloudera/sqoop/manager/SqlManager.java 1180266 > /src/java/com/cloudera/sqoop/mapreduce/DataDrivenImportJob.java 1180266 > /src/test/com/cloudera/sqoop/TestBoundaryQuery.java 1180266 > > Diff: https://reviews.apache.org/r/2270/diff > > > Testing > ------- > > All unit and thirdparty tests. > > > Thanks, > > Arvind > >
