----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4988/#review7581 -----------------------------------------------------------
Ship it! +1 Nemon, thanks for the patch. Noticed that the timestamp of the patch uploaded to the jira is 5/3 but on review board the last diff was made on 5/4. Nemon, would you mind uploading the latest patch to the jira as SQOOP-482-1.patch? - Kathleen On 2012-05-04 07:33:48, nemon lou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4988/ > ----------------------------------------------------------- > > (Updated 2012-05-04 07:33:48) > > > Review request for Sqoop, Jarek Cecho and Cheolsoo Park. > > > Summary > ------- > > I simply change the way the MySQLDumpMapper writes contents. > May be there are other ways to fix it.Please let me know,thank you! > > > This addresses bug SQOOP-482. > https://issues.apache.org/jira/browse/SQOOP-482 > > > Diffs > ----- > > ./src/java/org/apache/sqoop/mapreduce/MySQLDumpMapper.java 1332889 > > Diff: https://reviews.apache.org/r/4988/diff > > > Testing > ------- > > > Thanks, > > nemon > >