----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14240/ -----------------------------------------------------------
(Updated Oct. 4, 2013, 5:45 p.m.) Review request for Sqoop, Jarek Cecho, Jean-Marc Spaggiari, and Vasanth kumar RJ. Changes ------- Here's a new patch with updates addressing the latest round of changes. I also added a few unit tests to TestSqoopOptions. 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/mapreduce/ImportJobBase.java ab7f21e src/java/org/apache/sqoop/tool/BaseSqoopTool.java 0eca991 src/test/com/cloudera/sqoop/TestSqoopOptions.java 03e2504 Diff: https://reviews.apache.org/r/14240/diff/ Testing ------- Thanks, Alexandre Normand
