> On July 24, 2015, 2:16 a.m., Jarek Cecho wrote: > > src/java/org/apache/sqoop/mapreduce/MergeAvroMapper.java, line 52 > > <https://reviews.apache.org/r/35556/diff/2/?file=1019256#file1019256line52> > > > > Nit: whitespace
I have also fixed other similar problems. - Yibing ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35556/#review92755 ----------------------------------------------------------- On July 23, 2015, 10:02 a.m., Yibing Shi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35556/ > ----------------------------------------------------------- > > (Updated July 23, 2015, 10:02 a.m.) > > > Review request for Sqoop, Abraham Elmahrek and Jarek Cecho. > > > Repository: sqoop-trunk > > > Description > ------- > > SQOOP-1094: Add avro support to merge tool > > > Diffs > ----- > > src/java/org/apache/sqoop/mapreduce/AvroJob.java bb4755c > src/java/org/apache/sqoop/mapreduce/MergeAvroMapper.java PRE-CREATION > src/java/org/apache/sqoop/mapreduce/MergeAvroReducer.java PRE-CREATION > src/java/org/apache/sqoop/mapreduce/MergeJob.java 4e2a916 > src/java/org/apache/sqoop/mapreduce/MergeReducer.java cafff8a > src/java/org/apache/sqoop/mapreduce/MergeReducerBase.java PRE-CREATION > src/test/com/cloudera/sqoop/TestMerge.java 3821aa1 > > Diff: https://reviews.apache.org/r/35556/diff/ > > > Testing > ------- > > > Thanks, > > Yibing Shi > >
