sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Venkat Ranganathan" <n....@live.com>
Subject Re: Review Request 36042: Review Request for custom tool options for user defined plugins for sqoop(SQOOP-2333)
Date Sat, 08 Aug 2015 03:32:56 GMT

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

Ship it!


Thanks for updating the patch.   Will commit it after running unit tests

- Venkat Ranganathan


On Aug. 6, 2015, 12:33 a.m., Rakesh Sharma wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36042/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2015, 12:33 a.m.)
> 
> 
> Review request for Sqoop and Venkat Ranganathan.
> 
> 
> Bugs: SQOOP-2333
>     https://issues.apache.org/jira/browse/SQOOP-2333
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> -------
> 
> Sqoop allows users to develop their own plugins. But currently there is no way user plugins
can have their custom options. Users will have to use sqoop predefined options. This patch
adds support for a map to SqoopOptions which opens up possibilities for  users to add their
custom options as key/value pairs in the map.
> 
> 
> Diffs
> -----
> 
>   src/docs/dev/SqoopDevGuide.txt a81cfda 
>   src/docs/dev/plugin-arch.txt PRE-CREATION 
>   src/java/org/apache/sqoop/SqoopOptions.java 9405605 
>   src/java/org/apache/sqoop/util/SqoopJsonUtil.java PRE-CREATION 
>   src/test/org/apache/sqoop/util/TestSqoopJsonUtil.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/36042/diff/
> 
> 
> Testing
> -------
> 
> YES
> 
> 
> Thanks,
> 
> Rakesh Sharma
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message