----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61777/ -----------------------------------------------------------
(Updated Aug. 21, 2017, 1:15 p.m.) Review request for Sqoop. Changes ------- Based on codereview function is separated into different methods with meaningful names Bugs: SQOOP-3139 https://issues.apache.org/jira/browse/SQOOP-3139 Repository: sqoop-trunk Description ------- In the case if the database table name and split-by parameter was different (eg.: Mycol vs. mycol) Sqoop couldn't continue the query from the last value if the connection was broken. Diffs (updated) ----- src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java a78eb061 src/java/org/apache/sqoop/mapreduce/db/SQLServerDBRecordReader.java 9a3621b0 src/test/org/apache/sqoop/mapreduce/db/TestSQLServerDBRecordReader.java PRE-CREATION Diff: https://reviews.apache.org/r/61777/diff/2/ Changes: https://reviews.apache.org/r/61777/diff/1-2/ Testing ------- Thanks, Zoltán Tóth