-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2270/
-----------------------------------------------------------
(Updated 2011-10-07 23:04:25.276941)
Review request for Sqoop and Bilung Lee.
Changes
-------
Updating patch with review feedback.
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 (updated)
-----
/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