----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28220/#review62869 -----------------------------------------------------------
Ship it! Cool! Can you attach a patch to the Jira? - Abraham Elmahrek On Nov. 21, 2014, 1:38 a.m., Venkat Ranganathan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28220/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2014, 1:38 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1627 > https://issues.apache.org/jira/browse/SQOOP-1627 > > > Repository: sqoop-trunk > > > Description > ------- > > With the last few changes builds are broken with Hadoop100 and Hadoop20 > builds. There are a variety of issues. > We are using classes that are only in Hadoop 2 (TaskAtttemptContextImpl) - > This is not even public and stable. We should change it later! > We used Hadoop 2 specific methods because Hadoop 1 methods are deprecated > (Job.getInstance) > Method that is only available in later HBase versions (merge) for example > > I have updated the hadoop version to 1.0.4 to address the MutipleOutputs not > being present in Hadoop 1.0.0 and also updated hadoop20 profile to the latest > update which allows tests and code to compile (cdh3u5) > > > Diffs > ----- > > build.xml 21fd406 > src/java/org/apache/sqoop/mapreduce/HBaseImportJob.java a93114f > > src/test/org/apache/sqoop/mapreduce/mainframe/TestMainframeDatasetFTPRecordReader.java > 613ee7a > > src/test/org/apache/sqoop/mapreduce/mainframe/TestMainframeDatasetInputFormat.java > 70958e0 > > src/test/org/apache/sqoop/mapreduce/sqlserver/SqlServerUpsertOutputFormatTest.java > b8c4538 > > Diff: https://reviews.apache.org/r/28220/diff/ > > > Testing > ------- > > Ran with the profiles affected as well as 200 profile. With hadoop20 > profile we have 4 test suite failures in Parquet Import/Export and Accumulo > import/export and with hadoop100 profile, we have no failures. > > > Thanks, > > Venkat Ranganathan > >
