----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25562/#review54091 -----------------------------------------------------------
The patch no longer applies on top of SQOOP-1367 branch, could you rebase it Abe? - Jarek Cecho On Sept. 11, 2014, 9:58 p.m., Abraham Elmahrek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25562/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2014, 9:58 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1488 > https://issues.apache.org/jira/browse/SQOOP-1488 > > > Repository: sqoop-SQOOP-1367 > > > Description > ------- > > Run FROM and TO destroyers. > > > Diffs > ----- > > > execution/mapreduce/src/main/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngine.java > b05954b > execution/mapreduce/src/main/java/org/apache/sqoop/job/JobConstants.java > 4cdb002 > > execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopDestroyerExecutor.java > 59431f4 > > execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopFileOutputFormat.java > bbf7342 > > execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopNullOutputFormat.java > b3461bb > execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMapReduce.java > 2dfc487 > > test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableStagedRDBMSTest.java > cb782c7 > > Diff: https://reviews.apache.org/r/25562/diff/ > > > Testing > ------- > > Ran all tests and integration tests. > > > Thanks, > > Abraham Elmahrek > >
