----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34536/#review89705 -----------------------------------------------------------
Looks good to commit. Found a few checkstyle issues. Please fix them. Also, you have not closed the earlier issue after fixing it src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBInputFormat.java (line 91) <https://reviews.apache.org/r/34536/#comment142376> Minor Nit: No space after if. It violates the checkstyle rules. Same issue in the other similar checks src/java/org/apache/sqoop/mapreduce/db/IntegerSplitter.java (line 66) <https://reviews.apache.org/r/34536/#comment142377> Minor Checkstyle Nit: Need space after , in ,speedLimit src/java/org/apache/sqoop/mapreduce/db/IntegerSplitter.java (line 125) <https://reviews.apache.org/r/34536/#comment142378> Same as above - Venkat Ranganathan On June 28, 2015, 4:21 a.m., Rakesh Sharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34536/ > ----------------------------------------------------------- > > (Updated June 28, 2015, 4:21 a.m.) > > > Review request for Sqoop and Venkat Ranganathan. > > > Repository: sqoop-trunk > > > Description > ------- > > This is the review request for Apache jira : SQOOP-2334. > > > Diffs > ----- > > src/docs/man/import-args.txt 93f65ba > src/docs/user/import.txt df04157 > src/java/org/apache/sqoop/SqoopOptions.java ef6e0ce > src/java/org/apache/sqoop/config/ConfigurationConstants.java 2070b63 > src/java/org/apache/sqoop/config/ConfigurationHelper.java 8dc2061 > src/java/org/apache/sqoop/mapreduce/DataDrivenImportJob.java 7521464 > src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBInputFormat.java 2c59fe5 > src/java/org/apache/sqoop/mapreduce/db/DateSplitter.java 31e9351 > src/java/org/apache/sqoop/mapreduce/db/IntegerSplitter.java e6fefc6 > src/java/org/apache/sqoop/tool/BaseSqoopTool.java c97bb58 > src/java/org/apache/sqoop/tool/ImportTool.java c79e044 > src/test/org/apache/sqoop/mapreduce/db/TestIntegerSplitter.java 136afc7 > > Diff: https://reviews.apache.org/r/34536/diff/ > > > Testing > ------- > > YES > > > Thanks, > > Rakesh Sharma > >
