-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25326/
-----------------------------------------------------------
(Updated Sept. 4, 2014, 12:38 p.m.)
Review request for Sqoop.
Bugs: SQOOP-1491
https://issues.apache.org/jira/browse/SQOOP-1491
Repository: sqoop-trunk
Description (updated)
-------
This is a patch to SQOOP-1395. As the entity class name is no longer identical
to the record name in Avro schema, there is not need to have SqoopAvroRecord.
The patch proposes to do two things:
1. Remove SqoopAvroRecord (reverted ClassWriter to previous revision)
2. Move common Avro handling code to AvtoUtil
Diffs
-----
src/java/org/apache/sqoop/avro/AvroUtil.java 4b37d58
src/java/org/apache/sqoop/lib/SqoopAvroRecord.java 80875d2
src/java/org/apache/sqoop/mapreduce/AvroImportMapper.java 6adad79
src/java/org/apache/sqoop/mapreduce/DataDrivenImportJob.java 905ba8c
src/java/org/apache/sqoop/mapreduce/ParquetImportMapper.java effbadd
src/java/org/apache/sqoop/mapreduce/ParquetJob.java a74432a
src/java/org/apache/sqoop/orm/ClassWriter.java 4f9dedd
Diff: https://reviews.apache.org/r/25326/diff/
Testing
-------
All unittests passed. No new unittest added.
Manually tested couple of import tests.
Thanks,
Qian Xu
