----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33207/#review86562 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java <https://reviews.apache.org/r/33207/#comment138624> Take the constant from ActionStartXCommand. core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java <https://reviews.apache.org/r/33207/#comment138623> The code looks repetative, Same code present in ActionStartXCommand. Check if you can use it. - Jaydeep Vishwakarma On June 4, 2015, 6:40 a.m., Narayan Periwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33207/ > ----------------------------------------------------------- > > (Updated June 4, 2015, 6:40 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-2243 > https://issues.apache.org/jira/browse/OOZIE-2243 > > > Repository: oozie-git > > > Description > ------- > > Whenever the coord action or workflow gets killed only the job that has error > gets killed while the other hadoop jobs keeps on running and only the > workflow status gets changed to killed. > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 695853e > core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapperHelper.java > e22329d > > core/src/test/java/org/apache/oozie/command/wf/TestWorkflowKillXCommand.java > e493d4d > > hadooplibs/hadoop-utils-2/src/main/java/org/apache/oozie/action/hadoop/LauncherMainHadoopUtils.java > f6eda73 > > Diff: https://reviews.apache.org/r/33207/diff/ > > > Testing > ------- > > Done > > > Thanks, > > Narayan Periwal > >
