----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33624/#review84387 -----------------------------------------------------------
Ship it! Ship It! - Robert Kanter On May 18, 2015, 8:59 a.m., Shwetha GS wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33624/ > ----------------------------------------------------------- > > (Updated May 18, 2015, 8:59 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1993 > https://issues.apache.org/jira/browse/OOZIE-1993 > > > Repository: oozie-git > > > Description > ------- > > The issue was with picking up new execution path during re-runs. This will > also address the issue in OOZIE-1879. > Changes: > 1. SignalXCommand to pick up the new execution path > 2. Removed actionEndTimes comparison > 3. Aligned else ifs in LiteWorkflowInstance.signal(). No logic changes > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/WorkflowActionBean.java 6289158 > core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java 6138278 > core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java bccac51 > > core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionQueryExecutor.java > 0e99ae2 > core/src/main/java/org/apache/oozie/service/LiteWorkflowStoreService.java > d661d08 > core/src/main/java/org/apache/oozie/workflow/WorkflowLib.java ba5b6b1 > core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowInstance.java > 919c95a > core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowLib.java > 706cc2a > core/src/main/resources/oozie-log4j.properties 4b532ca > core/src/test/java/org/apache/oozie/command/wf/TestReRunXCommand.java > ce32e51 > core/src/test/resources/rerun-wf-fork.xml 8fa8f34 > > Diff: https://reviews.apache.org/r/33624/diff/ > > > Testing > ------- > > end to end test for the example workflow in the jira - with failures at > different nodes > > > Thanks, > > Shwetha GS > >
