-----------------------------------------------------------
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

Reply via email to