sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bilung Lee" <bleeapa...@gmail.com>
Subject Re: Review Request: SQOOP-735 Introduce output format option to Sqoop 2 import
Date Tue, 04 Dec 2012 16:29:09 GMT

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


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