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-650: Add ability to name mapreduce job generated by Sqoop


Copy link to this message
-
Re: Review Request: SQOOP-650: Add ability to name mapreduce job generated by Sqoop

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

(Updated Nov. 7, 2012, 12:13 a.m.)
Review request for Sqoop.
Changes
-------

Fixed indention issue.
Description
-------

I've added new extra parameter and made sure that it will be propagated to all places where Sqoop is creating mapreduce job.
This addresses bug SQOOP-650.
    https://issues.apache.org/jira/browse/SQOOP-650
Diffs (updated)
-----

  src/java/org/apache/sqoop/SqoopOptions.java 92c99962ae2276c1481657dfbeda4f14dfe7e8fc
  src/java/org/apache/sqoop/mapreduce/ExportJobBase.java b9eaa834ecacdfcac300827dc7e187aec862661a
  src/java/org/apache/sqoop/mapreduce/ImportJobBase.java efc2c373366c86a2fcb28290979e18781cbc8995
  src/java/org/apache/sqoop/tool/BaseSqoopTool.java 0c9cda610ccde6bd249f48fd438c88c94a239632
  src/java/org/apache/sqoop/tool/ExportTool.java 494887d8c973048c4e3f6003e4f83e5baf8cff50
  src/java/org/apache/sqoop/tool/ImportTool.java 415b315cf64bda65d110d60f33f3ee5cf2496292

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

I've tested it on real hadoop cluster. I did not included any test cases as I'm not sure how to actually test this automatically.
Thanks,

Jarek Cecho