----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4798/#review7570 -----------------------------------------------------------
Ship it! +1 Jarcec, Cheolsoo - sorry for not getting to this sooner. - Kathleen On 2012-04-19 05:56:56, Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4798/ > ----------------------------------------------------------- > > (Updated 2012-04-19 05:56:56) > > > Review request for Sqoop, Arvind Prabhakar and Cheolsoo Park. > > > Summary > ------- > > I've added code that is removing export directory in case that it's empty. > > (Recreating review on moved SVN repository) > > > This addresses bug SQOOP-443. > https://issues.apache.org/jira/browse/SQOOP-443 > > > Diffs > ----- > > /src/java/org/apache/sqoop/hive/HiveImport.java 1327832 > > Diff: https://reviews.apache.org/r/4798/diff > > > Testing > ------- > > ant -Dhadoopversion={20, 23, 100} test > real testing environment based on CDH3 > > > Thanks, > > Jarek > >
