----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1439/ -----------------------------------------------------------
(Updated 2011-08-10 22:51:19.745107) Review request for Sqoop. Changes ------- Thanks for the review, Arvind. Here's a new patch which addresses your comments. Summary ------- See https://issues.apache.org/jira/browse/SQOOP-305 This addresses bug SQOOP-305. https://issues.apache.org/jira/browse/SQOOP-305 Diffs (updated) ----- /src/java/com/cloudera/sqoop/manager/ConnManager.java 1156386 /src/java/com/cloudera/sqoop/mapreduce/AvroExportMapper.java PRE-CREATION /src/java/com/cloudera/sqoop/mapreduce/AvroImportMapper.java 1156386 /src/java/com/cloudera/sqoop/mapreduce/AvroInputFormat.java PRE-CREATION /src/java/com/cloudera/sqoop/mapreduce/AvroRecordReader.java PRE-CREATION /src/java/com/cloudera/sqoop/mapreduce/ExportJobBase.java 1156386 /src/java/com/cloudera/sqoop/mapreduce/JdbcExportJob.java 1156386 /src/java/com/cloudera/sqoop/orm/ClassWriter.java 1156386 /src/test/com/cloudera/sqoop/TestAvroExport.java PRE-CREATION /src/test/com/cloudera/sqoop/TestAvroImportExportRoundtrip.java PRE-CREATION /src/test/com/cloudera/sqoop/TestExport.java 1156386 /src/test/com/cloudera/sqoop/TestExportUpdate.java 1156386 /src/test/com/cloudera/sqoop/testutil/BaseSqoopTestCase.java 1156386 /src/test/com/cloudera/sqoop/testutil/ExportJobTestCase.java 1156386 Diff: https://reviews.apache.org/r/1439/diff Testing ------- Thanks, Tom
