----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4496/#review6547 -----------------------------------------------------------
Ship it! Thanks for the patch, Cheolsoo! - Bilung On 2012-03-28 02:46:54, Cheolsoo Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4496/ > ----------------------------------------------------------- > > (Updated 2012-03-28 02:46:54) > > > Review request for Sqoop, Arvind Prabhakar and Bilung Lee. > > > Summary > ------- > > BLOB is supported for text and sequence import only. > Provide further support for Avro import. > > > This addresses bug SQOOP-465. > https://issues.apache.org/jira/browse/SQOOP-465 > > > Diffs > ----- > > ./src/java/org/apache/sqoop/manager/ConnManager.java 1303136 > ./src/java/org/apache/sqoop/mapreduce/AvroImportMapper.java 1303098 > > Diff: https://reviews.apache.org/r/4496/diff > > > Testing > ------- > > Tested with Oracle XE 10g and MySQL 5.1 using newly added LobAvroImportTest > by SQOOP-470: https://reviews.apache.org/r/4527/ > > Ran ant test > Ran ant test -Dthirdparty=true > Ran ant checkstyle > > > Thanks, > > Cheolsoo > >
