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

(Updated Sept. 19, 2013, 11:35 p.m.)


Review request for Sqoop, Jarek Cecho and vasanthkumar.


Changes
-------

I uploaded the wrong diff. This one actually compiles.


Repository: sqoop-trunk


Description
-------

This is the patch to address some of the comments on Zhancheng Deng's review 
request (https://reviews.apache.org/r/13052/) which was basically the change to 
add the --bulk-load-dir option to support the HBase doBulkLoad function.


Diffs (updated)
-----

  src/java/org/apache/sqoop/SqoopOptions.java 01805f9 
  src/java/org/apache/sqoop/hbase/HBasePutProcessor.java 9ceb5bd 
  src/java/org/apache/sqoop/hbase/ToStringPutTransformer.java 5ccf311 
  src/java/org/apache/sqoop/manager/SqlManager.java 2a4992d 
  src/java/org/apache/sqoop/mapreduce/HBaseBulkImportJob.java PRE-CREATION 
  src/java/org/apache/sqoop/mapreduce/HBaseBulkImportMapper.java PRE-CREATION 
  src/java/org/apache/sqoop/tool/BaseSqoopTool.java ebb1857 

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


Testing
-------


Thanks,

Alexandre Normand

Reply via email to