Re: Review Request 35859: SQOOP-1493: Add ability to import/export true decimal in Avro instead of serializing it to String

[Abraham Elmahrek]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35859/
-----------------------------------------------------------

(Updated July 3, 2015, 3:28 a.m.)


Review request for Sqoop.


Bugs: SQOOP-1493
    https://issues.apache.org/jira/browse/SQOOP-1493


Repository: sqoop-trunk


Description
-------

commit 8583077a33956cd2a3abc05f73172210c31994a9
Author: Abraham Elmahrek <a...@apache.org>
Date:   Fri Jun 19 05:45:25 2015 +0300

    SQOOP-1493: Add ability to import/export true decimal in Avro instead of 
serializing it to String

:100644 100644 2e3d884... 628491c... M  ivy/libraries.properties
:100644 100644 ee3cf62... 5970981... M  
src/java/org/apache/sqoop/avro/AvroUtil.java
:100644 100644 d9569c5... 5650079... M  
src/java/org/apache/sqoop/manager/ConnManager.java
:100644 100644 20f056a... 76c3458... M  
src/java/org/apache/sqoop/mapreduce/AvroExportMapper.java
:100644 100644 aed1e72... a4ac46e... M  
src/java/org/apache/sqoop/mapreduce/AvroOutputFormat.java
:100644 100644 ab263c1... b60ee42... M  
src/java/org/apache/sqoop/mapreduce/GenericRecordExportMapper
:100644 100644 2576673... 8490582... M  
src/java/org/apache/sqoop/orm/AvroSchemaGenerator.java
:100644 100644 1c6f7f4... aff8d08... M  
src/java/org/apache/sqoop/orm/ClassWriter.java
:100644 100644 663828c... 42e36ab... M  
src/test/com/cloudera/sqoop/TestAvroExport.java
:100644 100644 260e80a... 2f680c8... M  
src/test/com/cloudera/sqoop/TestAvroImport.java


Diffs (updated)
-----

  LICENSE.txt c36c7ad0a24c9fde9e9099d96c28f3c0de07ccd3 
  ivy/libraries.properties 2e3d884ae172f4c501d6c7321c30fb62e0da5376 
  src/java/org/apache/sqoop/avro/AvroUtil.java 
ee3cf6214e45cf1f21be8c6cf6b486b9f35dcfbd 
  src/java/org/apache/sqoop/config/ConfigurationConstants.java 
e19c17b4a464a85262adc94ff976a3e8453089de 
  src/java/org/apache/sqoop/manager/ConnManager.java 
d9569c590884e46a434cc4bc35b4c2ba8ca5d345 
  src/java/org/apache/sqoop/mapreduce/AvroExportMapper.java 
20f056a9d5f51a044c40f0ff03b266eb16bc008f 
  src/java/org/apache/sqoop/mapreduce/AvroImportMapper.java 
0ea5ca410c6492044685efc6c55b45def27a24ed 
  src/java/org/apache/sqoop/mapreduce/AvroOutputFormat.java 
aed1e720aed984e310f309c562f109973c26822e 
  src/java/org/apache/sqoop/mapreduce/GenericRecordExportMapper.java 
ab263c166ce3a0462f09543055c888bea4b0058f 
  src/java/org/apache/sqoop/orm/AvroSchemaGenerator.java 
2576673bc2d39d6cb5240548d098e640cef8d6bf 
  src/java/org/apache/sqoop/orm/ClassWriter.java 
1c6f7f4e532be8d906e08d05b23cc5ce0e74f742 
  src/test/com/cloudera/sqoop/TestAvroExport.java 
663828c3cf118c83361a5ccc177ac63d9d4d4560 
  src/test/com/cloudera/sqoop/TestAvroImport.java 
260e80abefc43490e424a136287f31e84f229bf3 
  src/test/org/apache/sqoop/TestAvroImport.java PRE-CREATION 

Diff: https://reviews.apache.org/r/35859/diff/


Testing
-------

ant test && manual


Thanks,

Abraham Elmahrek