----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4988/#review7580 -----------------------------------------------------------
Ship it! Nemon's fix seems to be right to me. The write() method should be called only once per record. I also verified that his patch addresses the problem. @Nemon I am no commiter, so I cannot commit your patch. But you've got my vote! - Cheolsoo 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 > >
