Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop >> mail # dev >> Review Request 14240: SQOOP-1032: Add the --bulk-load-dir option to support the HBase doBulkLoad function


+
Alexandre Normand 2013-09-19, 23:21
+
Alexandre Normand 2013-09-19, 23:35
+
Alexandre Normand 2013-09-27, 23:01
+
Alexandre Normand 2013-09-28, 00:35
+
Alexandre Normand 2013-10-01, 20:30
+
Alexandre Normand 2013-10-02, 21:21
+
Alexandre Normand 2013-10-04, 17:45
+
Alexandre Normand 2013-10-09, 23:53
+
Alexandre Normand 2013-10-10, 16:46
+
Jarek Cecho 2013-10-10, 23:25
+
Jarek Cecho 2013-10-10, 16:36
Copy link to this message
-
Re: Review Request 14240: SQOOP-1032: Add the --bulk-load-dir option to support the HBase doBulkLoad function


> On Oct. 10, 2013, 4:36 p.m., Jarek Cecho wrote:
> > Indeed Alexandre, I believe we're very close to get it in! Thank you for all your effort! I've tested the patch on real cluster running HBase 0.94 and found one small issue noted below. I'll try to spin out HBase 0.95 to finish the testing on my side.

My apologies for the delay on finishing the testing against HBase 0.95. I've actually found an issue where the bulk load was failing on NullPointerException. I've discussed that with HBase guys and this particular problem will be fixed on HBase side via HBASE-9744. Big thanks to Jean-Daniel Cryans who was working with me on the issue!. I'll do last review and commit it.
- Jarek
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14240/#review26867
-----------------------------------------------------------
On Oct. 10, 2013, 4:46 p.m., Alexandre Normand wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14240/
> -----------------------------------------------------------
>
> (Updated Oct. 10, 2013, 4:46 p.m.)
>
>
> Review request for Sqoop, Jarek Cecho, Jean-Marc Spaggiari, and Vasanth kumar RJ.
>
>
> 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
> -----
>
>   src/docs/user/hbase-args.txt 8ba23eb
>   src/docs/user/hbase.txt 34f9875
>   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
>
>

+
Jarek Cecho 2013-10-09, 22:31
+
rj.vasanthkumar@... 2013-10-04, 09:45
+
Jarek Cecho 2013-10-01, 19:49
+
Alexandre Normand 2013-10-02, 20:40
+
Alexandre Normand 2013-10-03, 16:54
+
Alexandre Normand 2013-10-09, 00:09
+
Jarek Cecho 2013-10-08, 23:40
+
Jarek Cecho 2013-10-09, 21:01
+
Alexandre Normand 2013-10-09, 20:04