+
Jarek Cecho 2012-12-04, 01:01
-
Re: Review Request: SQOOP-735 Introduce output format option to Sqoop 2 importBilung Lee 2012-12-04, 16:29
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8332/#review14020
-----------------------------------------------------------
Ship it!
Ship It!
- Bilung Lee
On Dec. 4, 2012, 1:01 a.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8332/
> -----------------------------------------------------------
>
> (Updated Dec. 4, 2012, 1:01 a.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> I've added another enum type to support multiple output formats.
>
>
> This addresses bug SQOOP-735.
> https://issues.apache.org/jira/browse/SQOOP-735
>
>
> Diffs
> -----
>
> core/src/main/java/org/apache/sqoop/framework/configuration/OutputForm.java 3f70b5f47792de5e4f2bc5c11657f632cba1bae7
> core/src/main/java/org/apache/sqoop/framework/configuration/OutputFormat.java PRE-CREATION
> core/src/main/resources/framework-resources.properties 5ddf32debcdcc56d332f5316b4829eaf5903b379
> execution/mapreduce/src/main/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngine.java 4a5b305bbb628e47860e98c3dda8cbb28c1416e9
> execution/mapreduce/src/main/java/org/apache/sqoop/job/MapreduceExecutionError.java aa1f7332079c919d162bcf81ad01f35b7ca00386
>
> Diff: https://reviews.apache.org/r/8332/diff/
>
>
> Testing
> -------
>
> * Unit tests are passing
> * Tested on real cluster
>
>
> Thanks,
>
> Jarek Cecho
>
>
