----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29970/#review68551 -----------------------------------------------------------
Ship it! I've tried the patch on Hadoop profile 23, 100 and 200 and it worked there. Profile 20 seems to be so old, that some of the dependencies are not available even without this patch. - Jarek Cecho On Jan. 16, 2015, 7:08 p.m., Abraham Elmahrek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29970/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2015, 7:08 p.m.) > > > Review request for Sqoop, Hari Shreedharan, Jarek Cecho, and Venkat > Ranganathan. > > > Bugs: SQOOP-2024 > https://issues.apache.org/jira/browse/SQOOP-2024 > > > Repository: sqoop-trunk > > > Description > ------- > > commit 43cfe1f5b8ceb7d338ef522d252a2dcb2a3bb4e6 > Author: Abraham Elmahrek <[email protected]> > Date: Fri Jan 16 10:57:27 2015 -0800 > > SQOOP-2024: Hive import doesn't remove target directory in hive > > :100644 100644 07a354b... 097d622... M > src/java/org/apache/sqoop/hive/HiveImport.java > > > Diffs > ----- > > src/java/org/apache/sqoop/hive/HiveImport.java 07a354b > > Diff: https://reviews.apache.org/r/29970/diff/ > > > Testing > ------- > > ant test and manually verified via: sqoop import --connect > "jdbc:mysql://mysql.ent.cloudera.com/sqoop" --username sqoop --password sqoop > --table tbl --target-dir /tmp/test_tbls --hive-import > > > Thanks, > > Abraham Elmahrek > >
