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

Switch to Threaded View
Sqoop >> mail # dev >> Review Request: SQOOP-863 Sqoop2: Introduce ProgressThread into Extractor and Loader


Copy link to this message
-
Re: Review Request: SQOOP-863 Sqoop2: Introduce ProgressThread into Extractor and Loader

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

(Updated March 27, 2013, 12:40 a.m.)
Review request for Sqoop.
Changes
-------

I've applied Hari's suggestions.
Description
-------

I've ported ProgressThread from Sqoop1 and plug it into SqoopMapper and SqoopReducer.
This addresses bug SQOOP-863.
    https://issues.apache.org/jira/browse/SQOOP-863
Diffs (updated)
-----

  execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/ProgressRunnable.java PRE-CREATION
  execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopMapper.java 2a823032cfd91fb8fd1a8aee3ffd9d80c2e3bae6
  execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopReducer.java d2361482771eea1a34b14c767952c5592f89c45a

Diff: https://reviews.apache.org/r/9845/diff/
Testing
-------

Unit and integration tests seems to be passing. I've also verified the functionality on real cluster.
Thanks,

Jarek Cecho