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