You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sqoop.apache.org by Veena Basavaraj <vb...@cloudera.com> on 2014/11/13 18:58:49 UTC

Review Request 27994: Sqoop2: Use configurable writable to get Intermediate Data Format

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

Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira.

+ remove unused file.
+ add documentation on why the fromScheme and fromDataFormat are relevant for matching
+ add documentation on matching process before we write the data to the toDataFormat object


Diffs
-----

  execution/mapreduce/src/main/java/org/apache/sqoop/job/io/FieldTypes.java e96dc6e 
  execution/mapreduce/src/main/java/org/apache/sqoop/job/io/SqoopWritable.java ed118d2 
  execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopInputFormat.java 887b4bb 
  execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopMapper.java e25f404 
  execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopOutputFormatLoadExecutor.java 579101e 
  execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestSqoopWritable.java 3207e53 
  execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestSqoopOutputFormatLoadExecutor.java 67e965d 

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


Testing
-------


Thanks,

Veena Basavaraj