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

Switch to Plain View
Sqoop, mail # dev - Review Request: Sqoop2: Merge HdfsSequenceExportExtractor and HdfsTextExportExtractor to one Extractor


+
rj.vasanthkumar@... 2013-03-01, 09:43
+
Jarek Cecho 2013-03-17, 19:26
Copy link to this message
-
Re: Review Request: Sqoop2: Merge HdfsSequenceExportExtractor and HdfsTextExportExtractor to one Extractor
rj.vasanthkumar@... 2013-03-18, 07:30


> On March 17, 2013, 7:26 p.m., Jarek Cecho wrote:
> > Thank you Vasanth for working on this. I've done few tests on real cluster to verify the functionality and everything seems to be working just fine. I've noticed that you've even fixed reporting of exported rows, so thank you!
> >
> > Jarcec

Thank you Jarcec

Vasanth
- vasanthkumar
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9692/#review18044
-----------------------------------------------------------
On March 1, 2013, 9:43 a.m., vasanthkumar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9692/
> -----------------------------------------------------------
>
> (Updated March 1, 2013, 9:43 a.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> Merge HdfsSequenceExportExtractor and HdfsTextExportExtractor to one HdfsExportExtractor
>
>
> This addresses bug SQOOP-783.
>     https://issues.apache.org/jira/browse/SQOOP-783
>
>
> Diffs
> -----
>
>   execution/mapreduce/src/main/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngine.java b201a8d
>   execution/mapreduce/src/main/java/org/apache/sqoop/job/etl/HdfsExportExtractor.java PRE-CREATION
>   execution/mapreduce/src/main/java/org/apache/sqoop/job/etl/HdfsSequenceExportExtractor.java 2280828
>   execution/mapreduce/src/main/java/org/apache/sqoop/job/etl/HdfsTextExportExtractor.java ae419ff
>   execution/mapreduce/src/test/java/org/apache/sqoop/job/TestHdfsExtract.java fae6573
>
> Diff: https://reviews.apache.org/r/9692/diff/
>
>
> Testing
> -------
>
> Passing all tests.
>
>
> Thanks,
>
> vasanthkumar
>
>