> On 2012-05-04 16:58:55, Kathleen Ting wrote: > > +1 > > Jarcec, Cheolsoo - sorry for not getting to this sooner. > > Kathleen Ting wrote: > The timestamp of the patch uploaded to the jira is 2/18 but on review > board I see a comment made at 4/19 stating "Patch rebase to current moved > repository trunk." Jarcec, is the latest patch attached to the jira? >
Good point Kate, thanks for checking, I've uploaded rebased version to JIRA. Jarcec - Jarek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4798/#review7570 ----------------------------------------------------------- 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 > >
