> On 2012-05-04 06:44:10, Cheolsoo Park wrote: > > This patch has been posted for a while. It would be nice if someone could > > commit this patch. > > > > The jira SQOOP-483 will be likely to touch the same area of code, so it > > will be nice if we can avoid any merge conflicts.
Hi Cheolsoo, thank you very much for your review! However I believe that we have the "two committer" policy in sqoop, so that I'm not allowed to commit my own patch :-( Jarcec - Jarek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4798/#review7548 ----------------------------------------------------------- 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 > >
