----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68237/ -----------------------------------------------------------
(Updated Sept. 7, 2018, 8:08 a.m.) Review request for oozie, Andras Salamon, Attila Sasvari, Kinga Marton, and Peter Bacsko. Changes ------- Addressing review comments. Repository: oozie-git Description ------- OOZIE-3298 OYA: external ID is not filled properly and failing MR job is treated as SUCCEEDED Diffs (updated) ----- core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java 05fac3953eed5a5e2a16fa362656596f3a962a88 core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java 83a23f5220aa72ba15edc8b98ef80a74213fcee8 core/src/test/java/org/apache/oozie/action/hadoop/ActionExecutorTestCase.java f39bba2c691435354dac6da7794e5142b511d937 core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java a31079a41d30677d35a253a4a69505c21aa585f6 core/src/test/java/org/apache/oozie/action/hadoop/TestYarnApplicationIdComparator.java PRE-CREATION core/src/test/java/org/apache/oozie/action/hadoop/TestYarnApplicationIdFinder.java PRE-CREATION core/src/test/java/org/apache/oozie/action/oozie/TestSubWorkflowActionExecutor.java 893405e38ad71d22460768b76ed03ac4d9a0b95d sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java c9e2a914a4b88640ac1c370b8971355dc087a235 sharelib/streaming/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java f460b6bd11f60dfb397c6bba82be1427c2d1b570 Diff: https://reviews.apache.org/r/68237/diff/10/ Changes: https://reviews.apache.org/r/68237/diff/9-10/ Testing ------- Tested on a real cluster, plus added test cases to `TestMapReduceActionExecutor` and new test classes. Thanks, András Piros
