----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2270/#review2442 -----------------------------------------------------------
/src/java/com/cloudera/sqoop/mapreduce/DataDrivenImportJob.java <https://reviews.apache.org/r/2270/#comment5551> I could do that but then I will have to do a null check in DataDrivenDBInputFormat. Otherwise, the following code will raise a NPE. Hence I kept it separate. If ou think that is better, I can certainly change that. Please let me know. - Arvind On 2011-10-07 02:31:15, Arvind Prabhakar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2270/ > ----------------------------------------------------------- > > (Updated 2011-10-07 02:31:15) > > > 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 1178856 > /src/java/com/cloudera/sqoop/mapreduce/DataDrivenImportJob.java 1178856 > /src/test/com/cloudera/sqoop/TestBoundaryQuery.java 1178856 > > Diff: https://reviews.apache.org/r/2270/diff > > > Testing > ------- > > All unit and thirdparty tests. > > > Thanks, > > Arvind > >
