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

Review request for Sqoop and Arvind Prabhakar.


Summary
-------

This is not fully featured patch yet, it's more only preview of what I have in 
my mind when I created the bug and how would I image to solve it. I would like 
to check with community whether this is acceptable solution and if so, I'll 
finish the patch.

Things that are missing and I'll add them if this way will be accepted:
* Tests
* Documentation
* Supporting for type names (so that user don't have to type the integer 
constants on command line)

Any feedback will be greatly appreciated.


This addresses bug sqoop-342.
    https://issues.apache.org/jira/browse/sqoop-342


Diffs
-----

  /src/java/com/cloudera/sqoop/SqoopOptions.java 1172203 
  /src/java/com/cloudera/sqoop/hive/HiveTypes.java 1172203 
  /src/java/com/cloudera/sqoop/hive/TableDefWriter.java 1172203 
  /src/java/com/cloudera/sqoop/manager/ConnManager.java 1172203 
  /src/java/com/cloudera/sqoop/manager/SqlManager.java 1172203 
  /src/java/com/cloudera/sqoop/mapreduce/JdbcExportJob.java 1172203 
  /src/java/com/cloudera/sqoop/orm/ClassWriter.java 1172203 
  /src/java/com/cloudera/sqoop/tool/BaseSqoopTool.java 1172203 

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


Testing
-------


Thanks,

Jarek

Reply via email to