----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4075/#review5370 -----------------------------------------------------------
Ship it! Thanks for the patch! - Bilung On 2012-02-27 23:28:35, Cheolsoo Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4075/ > ----------------------------------------------------------- > > (Updated 2012-02-27 23:28:35) > > > Review request for Sqoop and Bilung Lee. > > > Summary > ------- > > The call to ResultSet.isLast() should be replaced by ResultSet.isAfterLast() > where necessary in order to help support Sybase for data import. > > > This addresses bug SQOOP-425. > https://issues.apache.org/jira/browse/SQOOP-425 > > > Diffs > ----- > > ./src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java 1294374 > > Diff: https://reviews.apache.org/r/4075/diff > > > Testing > ------- > > Ran ant test > > > Thanks, > > Cheolsoo > >
