----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11680/#review21658 -----------------------------------------------------------
looks like the patch created by 'svn diff' is not including any of the moved files in the patch. Also, if I modify a moved file, its including the modifications, but because it doesn't include the original moved file, RB won't even accept it. It looks like svn 1.7 might have a fix for this that I'll try, but in the mean time, the patch generated by git on the JIRA is better to look at. - Robert Kanter On June 7, 2013, 8:21 p.m., Robert Kanter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11680/ > ----------------------------------------------------------- > > (Updated June 7, 2013, 8:21 p.m.) > > > Review request for oozie. > > > Description > ------- > > The patch is mostly moving some files from core to the Oozie sharelib. There > was some minor refactoring to fix some issues with tests. Otherwise, the only > actual code changes was to add the option to use the launcher jar or not > (oozie.action.use.launcher.jar). > > > This addresses bug OOZIE-1315. > https://issues.apache.org/jira/browse/OOZIE-1315 > > > Diffs > ----- > > trunk/core/pom.xml 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionStats.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionType.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/FSLauncherURIHandler.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/HCatLauncherURIHandler.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/HiveActionExecutor.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherException.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapperHelper.java > PRE-CREATION > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherURIHandler.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherURIHandlerFactory.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MRStats.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceMain.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/PipesMain.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/PrepareActionsDriver.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ShellMain.java > 1490336 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/SqoopActionExecutor.java > 1490336 > trunk/core/src/main/java/org/apache/oozie/util/HCatURI.java 1490336 > trunk/core/src/main/resources/oozie-default.xml 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestFSPrepareActions.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHCatPrepareActions.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java > 1490336 > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestLauncher.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionError.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutorUberJar.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestPrepareActionsDriver.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestShellActionExecutor.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionCheckXCommand.java > 1490336 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionStartXCommand.java > 1490336 > trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java > 1490336 > trunk/core/src/test/java/org/apache/oozie/util/TestHCatURI.java 1490336 > trunk/core/src/test/resources/hsqldb-oozie-site.xml 1490336 > trunk/examples/pom.xml 1490336 > trunk/examples/src/test/resources/hsqldb-oozie-site.xml 1490336 > trunk/minitest/src/test/resources/hsqldb-oozie-site.xml 1490336 > trunk/pom.xml 1490336 > trunk/sharelib/distcp/pom.xml 1490336 > trunk/sharelib/hcatalog/pom.xml 1490336 > trunk/sharelib/hive/pom.xml 1490336 > > trunk/sharelib/hive/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java > 1490336 > trunk/sharelib/oozie/pom.xml 1490336 > > trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java > PRE-CREATION > > trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/MapReduceMain.java > PRE-CREATION > trunk/sharelib/pig/pom.xml 1490336 > > trunk/sharelib/pig/src/test/java/org/apache/oozie/action/hadoop/TestPigActionExecutor.java > 1490336 > trunk/sharelib/sqoop/pom.xml 1490336 > > trunk/sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java > 1490336 > trunk/sharelib/streaming/pom.xml 1490336 > trunk/tools/src/test/resources/hsqldb-oozie-site.xml 1490336 > trunk/webapp/pom.xml 1490336 > > Diff: https://reviews.apache.org/r/11680/diff/ > > > Testing > ------- > > - Unit tests for oozie.action.use.launcher.jar > - I verified that it works correctly with oozie.action.use.launcher.jar set > to true and false. > > > Thanks, > > Robert Kanter > >
