sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Cecho" <jar...@apache.org>
Subject Re: Review Request: SQOOP-443: Calling sqoop with hive import is not working multiple times due to kept output directory
Date Fri, 04 May 2012 06:48:47 GMT


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message