----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28637/#review63970 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/action/oozie/SubWorkflowActionExecutor.java <https://reviews.apache.org/r/28637/#comment106345> this method looks like common can be move it to some common place, it has nothing specific to do with JavaactionExecutor. - Jaydeep Vishwakarma On Dec. 3, 2014, 9:20 a.m., shwethags wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28637/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2014, 9:20 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-2030 > https://issues.apache.org/jira/browse/OOZIE-2030 > > > Repository: oozie-git > > > Description > ------- > > 1. Propagate properties from configuration and job-xml from parent workflow > to sub-workflow. Currently, its propagated as workflow properties, but they > are required in action conf > 2. For java actions, add properties from configuration and job-xml to > launcher job > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > f207d74 > > core/src/main/java/org/apache/oozie/action/oozie/SubWorkflowActionExecutor.java > bda34b5 > > core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java > 48166a5 > > core/src/test/java/org/apache/oozie/action/oozie/TestSubWorkflowActionExecutor.java > 0d7e926 > examples/src/main/apps/subwf/job.xml PRE-CREATION > examples/src/main/apps/subwf/workflow.xml 9873a50 > > Diff: https://reviews.apache.org/r/28637/diff/ > > > Testing > ------- > > UTs and end to end test > > > Thanks, > > shwethags > >
