----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62936/ -----------------------------------------------------------
(Updated Oct. 16, 2017, 4:19 p.m.) Review request for oozie, Attila Sasvari, Peter Cseh, and Peter Bacsko. Changes ------- Addressing review comments. Repository: oozie-git Description ------- OOZIE-2896 Ensure compatibility for existing LauncherMapper settings Diffs (updated) ----- core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java 76d0daa4f7528de6fb3c8993bae4dd28b6f7865b core/src/main/java/org/apache/oozie/action/hadoop/LauncherConfigurationInjector.java PRE-CREATION core/src/main/java/org/apache/oozie/service/ConfigurationService.java a51933022c14ea4ce06a65cc3c123bed8a8f7f34 core/src/main/resources/oozie-default.xml 2389b993b7b1769521d6dd0c7386e99e7516210a core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java a7bd357f8938d148a150db43135a7a8fe7f0da4c core/src/test/java/org/apache/oozie/action/hadoop/TestLauncherConfigurationInjector.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java ee1a32ac9b071009e1dbe78396520d778f58bddc Diff: https://reviews.apache.org/r/62936/diff/2/ Changes: https://reviews.apache.org/r/62936/diff/1-2/ Testing ------- JUnit tests: * `TestJavaActionExecutor` * `TestLauncherConfigurationFilter` Functional tests: submit and run `examples/apps/java-main/workflow.xml` in three modes: * only `oozie.launcher.` parameters are present: `oozie.launcher.` parameters are passed through * `oozie.launcher.` and override (`yarn.`, `mapreduce.`, and `mapred.`) parameters are also present: `oozie.launcher.` parameters are passed through * only override (`yarn.`, `mapreduce.`, and `mapred.`) parameters are present: override parameters are passed through Thanks, András Piros